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

Re: [PATCH] Better progress indicator for the 'svn' command-line client

From: Mukund <mukund_at_tessna.com>
Date: 2003-07-09 15:12:58 CEST

On Wed, Jul 09, 2003 at 02:05:35PM +0100, Philip Martin wrote:
| Since there is only one integer, the callback itself must have state
| information to count the number of times it is called. Is this the
| best way to do it? Perhaps the state should be in the calling
| library, with two numbers being passed to the callback.

The feature is quite a colourful bikeshed already :-)

I cannot find advantages or disadvantages of either approach.
An argument would be that since all clients would need to have a counter
to show a progress, why not implement this in the client library.
A counter argument would be since all clients do not need to show
progress they don't need a counter, and it needn't be maintained
unnecessarily. Yet another colourful bikeshed. Submit your patch
whichever way you want it :-) Yesterday I was going to add code to
display time taken...

Mukund

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Wed Jul 9 15:16:16 2003

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

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.