You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@subversion.apache.org by Julian Foad <ju...@wandisco.com> on 2010/08/25 10:55:47 UTC
Re: svn commit: r988627 - in /subversion/trunk/subversion:
bindings/javahl/native/ include/ libsvn_client/ svn/
On Tue, 2010-08-24, hwright@apache.org wrote:
> Log:
> Move the commit callback out of the client context and into the individual APIs
> which require it.
Hurrah!
Reviewed and looks good.
- Julian
> * subversion/bindings/javahl/native/SVNClient.cpp
> (remove, commit, copy, move, mkdir, doImport, propertySet): Set the callback
> as a param, rather than in the client context.
>
> * subversion/svn/propdel-cmd.c,
> subversion/svn/move-cmd.c,
> subversion/svn/mkdir-cmd.c,
> subversion/svn/copy-cmd.c,
> subversion/svn/commit-cmd.c,
> subversion/svn/propset-cmd.c,
> subversion/svn/delete-cmd.c,
> subversion/svn/import-cmd.c,
> subversion/svn/propedit-cmd.c:
> Update API consumers to provide the commit callback.
>
> * subversion/svn/main.c
> (main): Don't set the commit callback context members.
>
> * subversion/include/svn_client.h
> (svn_client_ctx_t): Remove the commit callback and baton members.
> (svn_client_mkdir4, svn_client_delete4, svn_client_import4,
> svn_client_commit5, svn_client_copy6, svn_client_move6, svn_client_propset4):
> Add commit callback parameters.
>
> * subversion/libsvn_client/deprecated.c
> (capture_baton_t): Remove unneded members.
> (capture_commit_info): Just dup the commit info and exit.
> (svn_client_mkdir3, svn_client_import3, svn_client_delete3,
> svn_client_commit4, svn_client_copy5, svn_client_move5, svn_client_propset3):
> Remove the use of the shadow baton.
>
> * subversion/libsvn_client/delete.c,
> subversion/libsvn_client/prop_commands.c,
> subversion/libsvn_client/copy.c,
> subversion/libsvn_client/add.c,
> subversion/libsvn_client/commit.c:
> Explicitly pass around the commit callback and baton, rather than relying
> on the client context.
[...]
Re: svn commit: r988627 - in /subversion/trunk/subversion:
bindings/javahl/native/ include/ libsvn_client/ svn/
Posted by "Hyrum K. Wright" <hy...@mail.utexas.edu>.
Thanks!
On Wed, Aug 25, 2010 at 6:55 AM, Julian Foad <ju...@wandisco.com> wrote:
> On Tue, 2010-08-24, hwright@apache.org wrote:
>> Log:
>> Move the commit callback out of the client context and into the individual APIs
>> which require it.
>
> Hurrah!
>
> Reviewed and looks good.
>
> - Julian
>
>
>> * subversion/bindings/javahl/native/SVNClient.cpp
>> (remove, commit, copy, move, mkdir, doImport, propertySet): Set the callback
>> as a param, rather than in the client context.
>>
>> * subversion/svn/propdel-cmd.c,
>> subversion/svn/move-cmd.c,
>> subversion/svn/mkdir-cmd.c,
>> subversion/svn/copy-cmd.c,
>> subversion/svn/commit-cmd.c,
>> subversion/svn/propset-cmd.c,
>> subversion/svn/delete-cmd.c,
>> subversion/svn/import-cmd.c,
>> subversion/svn/propedit-cmd.c:
>> Update API consumers to provide the commit callback.
>>
>> * subversion/svn/main.c
>> (main): Don't set the commit callback context members.
>>
>> * subversion/include/svn_client.h
>> (svn_client_ctx_t): Remove the commit callback and baton members.
>> (svn_client_mkdir4, svn_client_delete4, svn_client_import4,
>> svn_client_commit5, svn_client_copy6, svn_client_move6, svn_client_propset4):
>> Add commit callback parameters.
>>
>> * subversion/libsvn_client/deprecated.c
>> (capture_baton_t): Remove unneded members.
>> (capture_commit_info): Just dup the commit info and exit.
>> (svn_client_mkdir3, svn_client_import3, svn_client_delete3,
>> svn_client_commit4, svn_client_copy5, svn_client_move5, svn_client_propset3):
>> Remove the use of the shadow baton.
>>
>> * subversion/libsvn_client/delete.c,
>> subversion/libsvn_client/prop_commands.c,
>> subversion/libsvn_client/copy.c,
>> subversion/libsvn_client/add.c,
>> subversion/libsvn_client/commit.c:
>> Explicitly pass around the commit callback and baton, rather than relying
>> on the client context.
> [...]
>
>