On Mon, 01 Oct 2007, Eric Gillespie wrote:
> Jack Repenning <email@example.com> writes:
> > On Oct 1, 2007, at 6:32 PM, Eric Gillespie wrote:
> > > I advocate setting line buffering in
> > > svn_cmdline_init, something like:
> > >
> > > setvbuf(stderr, NULL, _IOLBF, 0);
> > > setvbuf(stdout, NULL, _IOLBF, 0);
> > +1 on stdout. But isn't stderr ordinarily character-buffered, for
> > reasons having to do with ensuring you can report program crash even
> > if the crash is because you can't allocate a line buffer to report it?
> When connected to a tty, yes; good catch. We should use _IONBF
> for stderr.
Patch attached. Rather than operating on stderr, svn_cmdline_init()
takes an ERROR_STREAM parameter. Return values from setvbuf() appear
to be best-off ignored, I suppose.
Received on Tue Oct 2 05:21:47 2007
- text/plain attachment: patch
- application/pgp-signature attachment: stored