On Tue, 2010-08-24, hwri...@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. [...]