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

Re: svn commit: r28567 - in branches/tree-conflicts/subversion: include libsvn_wc svn tests/cmdline

From: Charles Acknin <charlesacknin_at_gmail.com>
Date: 2007-12-20 02:45:23 CET

On Dec 20, 2007 1:45 AM, Stefan Sperling <stsp@elego.de> wrote:
> So how can we get rid of the misleading comment you quoted?
>
> A better solution may be to simply not make "-t" imply "-v"
> (which is wrong anyway from the UI perspective) and call
> svn_client_status3() like this instead:
>
> SVN_ERR(svn_client_status3(&repos_rev, target, rev,
> print_status, status_baton,
> opt_state->depth,
> get_all ----> (opt_state->verbose
> get_all continued ---> || opt_state->tree_conflicted),
> opt_state->update,
> opt_state->no_ignore,
> opt_state->ignore_externals,
> ctx, pool));
>
> We could then remove the misleading comment in main.c
> that caused this whole mail in the first place :)

Thanks for taking me through the whole process! Yes I think that
looks much better and less confusing this way.
(And don't hesitate to propedit svn:log as hwright suggested :-)

Charles

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Thu Dec 20 02:45:34 2007

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