Re: [PATCH] Remove redundant svn_error_return() wrapper

2010-11-18 Thread Noorul Islam K M
Stefan Sperling writes: > Hi Noorul, > > two small remarks below: > > On Thu, Nov 18, 2010 at 04:51:50PM +0530, Noorul Islam K M wrote: >> Index: subversion/svn/log-cmd.c >> === >> --- subversion/svn/log-cmd.c (revision 1036324) >> +

Re: [PATCH] Remove redundant svn_error_return() wrapper

2010-11-18 Thread Julian Foad
Thanks Stefan; I already spotted these and tweaked it before commit. - Julian On Thu, 2010-11-18 at 13:00 +0100, Stefan Sperling wrote: > Hi Noorul, > > two small remarks below: > > On Thu, Nov 18, 2010 at 04:51:50PM +0530, Noorul Islam K M wrote: > > Index: subversion/svn/log-cmd.c > > ==

Re: [PATCH] Remove redundant svn_error_return() wrapper

2010-11-18 Thread Stefan Sperling
Hi Noorul, two small remarks below: On Thu, Nov 18, 2010 at 04:51:50PM +0530, Noorul Islam K M wrote: > Index: subversion/svn/log-cmd.c > === > --- subversion/svn/log-cmd.c (revision 1036324) > +++ subversion/svn/log-cmd.c (workin

Re: [PATCH] Remove redundant svn_error_return() wrapper

2010-11-18 Thread Julian Foad
Noorul Islam K M wrote: > Remove redundant svn_error_return() wrapper Thanks, Noorul. Committed in r1036419, except for resolved-cmd.c which I had already committed in r1036383. - Julian

[PATCH] Remove redundant svn_error_return() wrapper

2010-11-18 Thread Noorul Islam K M
Log [[[ Remove redundant svn_error_return() wrapper * subversion/svn/patch-cmd.c, subversion/svn/revert-cmd.c, subversion/svn/log-cmd.c, subversion/svn/changelist-cmd.c, subversion/svn/resolved-cmd.c, subversion/svn/cleanup-cmd.c, subversion/svn/upgrade-cmd.c, subversion/svn/add-c