[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: [PATCH] Flush stdout more often

From: Vincent Lefevre <vincent+svn_at_vinc17.org>
Date: 2006-03-09 02:40:05 CET

On 2006-03-08 08:27:29 +0100, Stefan Haller wrote:
> OK, in that case I see two possibilities:
> - add the flushing to libsvn_client/diff.c in the hope that it is the
> appropriate thing for all clients, not just the svn command line client
> (which I tend to think is true; at least I can't imagine that it would
> hurt anybody); or

Yes, there could even be some benefits. In general, it is better to
flush if one knows that the following output won't be sent before
some time.

> - live with the fact that diff output doesn't have the level of
> streaminess that would be nice to have. Diff would then be no worse
> than it is now, but all other subcommands would be improved; to me this
> improvement would outweigh the inconsistency.

Yes, and IMHO, since diff often generates more output, the problem
is less important with diff.

Also, stdout and stderr should be synchronized. This is currently not
the case due to the lack of flush. For instance, I currently get:

dixsept:~/wd> svn info -r10000 tex www | grep '^URL'
Connected to ay (from
Connected to ay (from
Connected to ay (from
Connected to ay (from
URL: svn+ssh://home/home/lefevre/svn/tex
URL: svn+ssh://home/home/lefevre/svn/www

instead of

Connected to ay (from
Connected to ay (from
URL: svn+ssh://home/home/lefevre/svn/tex
Connected to ay (from
Connected to ay (from
URL: svn+ssh://home/home/lefevre/svn/www

Vincent Lefèvre <vincent_at_vinc17.org> - Web: <http://www.vinc17.org/>
100% accessible validated (X)HTML - Blog: <http://www.vinc17.org/blog/>
Work: CR INRIA - computer arithmetic / SPACES project at LORIA
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Thu Mar 9 02:40:29 2006

This is an archived mail posted to the Subversion Dev mailing list.