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

Posted to dev@subversion.apache.org by Greg Stein <gs...@gmail.com> on 2012/05/12 06:12:09 UTC

Re: svn commit: r1337392 - /subversion/trunk/subversion/libsvn_client/commit_util.c

On Fri, May 11, 2012 at 5:47 PM,  <rh...@apache.org> wrote:
>...
> +++ subversion/trunk/subversion/libsvn_client/commit_util.c Fri May 11 21:47:21 2012
>...
> +static svn_error_t *
> +harvest_not_present_for_copy(struct harvest_baton *baton,
> +                             const char *local_abspath,
> +                             const char *repos_root_url,
> +                             const char *commit_relpath,
> +                             apr_pool_t *scratch_pool)

This function doesn't need the whole BATON. There are only five values
needed (and one is RESULT_POOL which is normal dual-pool design rather
than "another argument"). Another is WC_CTX which is also kind of
invisible. So you're really just swapping out BATON for
CHECK_URL_FUNC/BATON and COMMITTABLES.

>...

Cheers,
-g