You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@subversion.apache.org by Daniel Rall <dl...@collab.net> on 2007/09/28 16:26:33 UTC

Re: svn commit: r26825 - trunk/subversion/libsvn_client

On Fri, 28 Sep 2007, kameshj@tigris.org wrote:

> Author: kameshj
> Date: Fri Sep 28 03:06:54 2007
> New Revision: 26825
> 
> Log:
> Code cleanup.
> 
> * subversion/libsvn_client/merge.c
>   (do_single_file_merge): Accept 'notify_b' from callers.
>   (svn_client_merge3, svn_client_merge_peg3): Adjust for the above change.

It'd be nice if this log message was more explicit about indicating
that the callers were already stack-allocating the notify baton.

RE: svn commit: r26825 - trunk/subversion/libsvn_client

Posted by Kamesh Jayachandran <ka...@collab.net>.

>> * subversion/libsvn_client/merge.c
>>   (do_single_file_merge): Accept 'notify_b' from callers.
>>   (svn_client_merge3, svn_client_merge_peg3): Adjust for the above change.

>It'd be nice if this log message was more explicit about indicating
>that the callers were already stack-allocating the notify baton.


Dan,

Thanks, updated the log message.

With regards
Kamesh Jayachandran