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 2008/09/30 18:22:17 UTC
Re: svn commit: r33350 - trunk/subversion/libsvn_repos
On Mon, Sep 29, 2008 at 10:58 AM, <hw...@tigris.org> wrote:
>...
> +++ trunk/subversion/libsvn_repos/load.c Mon Sep 29 10:58:09 2008 (r33350)
>...
> +static svn_repos_parser_fns_t *
> +fns_from_fns2(const svn_repos_parse_fns2_t *fns2,
> + apr_pool_t *pool)
Why this function? It appears there is just one caller, so why not
simply do this inline?
Cheers,
-g
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Re: svn commit: r33350 - trunk/subversion/libsvn_repos
Posted by "Hyrum K. Wright" <hy...@mail.utexas.edu>.
Greg Stein wrote:
> On Mon, Sep 29, 2008 at 10:58 AM, <hw...@tigris.org> wrote:
>> ...
>
>> +++ trunk/subversion/libsvn_repos/load.c Mon Sep 29 10:58:09 2008 (r33350)
>> ...
>> +static svn_repos_parser_fns_t *
>> +fns_from_fns2(const svn_repos_parse_fns2_t *fns2,
>> + apr_pool_t *pool)
>
> Why this function? It appears there is just one caller, so why not
> simply do this inline?
In this instance, I was just following the model of the fns2_from_fns()
function, which also is only called once. The compile will do the inlining for
us, and I think that the functions provide useful abstractions, so I'm inclined
to leave it as is. (Though if others want to change it, that's fine by me.)
-Hyrum