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

On Dec 19, 2007 6:44 PM, <stsp@tigris.org> wrote:
> @@ -1426,6 +1428,12 @@
> err = svn_utf_cstring_to_utf8(&path_utf8, opt_arg, pool);
> opt_state.from_source = svn_path_canonicalize(path_utf8, pool);
> break;
> + case 't':
> + opt_state.show_tree_conflicts = TRUE;
> + /* '-t' implies '-v', otherwise svn_wc_client3() won't return
> + the entries we need. This hack isn't user-visible. */
> + opt_state.verbose = TRUE;
> + break;

Shouldn't we let the user know that '-t' implies '-v'? Because in the
end, he's going to see the extra stuff that '-v' prints, right?
I similarly fixed 'svn di --svnpatch' to imply --no-diff-deleted
earlier today, and I thought it was fair to comment it in 'svn di -h'
too.

Charles

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Wed Dec 19 23:05:06 2007