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

Re: svn commit: rev 3429 - trunk/subversion/libsvn_client trunk/subversion/clients/cmdline

From: Greg Stein <gstein_at_lyra.org>
Date: 2002-10-22 00:17:55 CEST

On Mon, Oct 21, 2002 at 01:12:10PM -0500, kfogel@tigris.org wrote:
>...
> * subversion/libsvn_client/add.c
> (svn_client_add): Don't wrap the error; the original error message
> contains the path, without which the warning is quite obscure.
> Having the original error be a child of the new error didn't help,
> because the error eventually reaches svn_handle_warning, not
> svn_handle_error. Also, remove unnecessary parens and reformat to
> 80 columns.
>...
> - if ((kind == svn_node_dir) && (recursive))
> - err = add_dir_recursive (path, adm_access, notify_func, notify_baton, pool);
> + if ((kind == svn_node_dir) && recursive)

Heh. You didn't remove all the unnecessary parens :-)

/me continues to wonder why people think that == doesn't bind tighter than
&& or || ... Maybe too many LISPers and they're trying to apply their
bad^H^H^HLISP habits to SVN...

:-)

Cheers,
-g

-- 
Greg Stein, http://www.lyra.org/
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Tue Oct 22 00:19:03 2002

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.