Re: svn_error_t *make_foo(svn_foo_t **foo_p, apr_pool_t *result, apr_pool_t *scratch);

On Tue, Sep 13, 2011 at 09:39:11AM -0500, Hyrum K Wright wrote:
> On Tue, Sep 13, 2011 at 9:02 AM, C. Michael Pilato <cmpilato_at_collab.net> wrote:
> > On 09/12/2011 09:12 PM, Daniel Shahaf wrote:
> >> [RFC] In new code, rename RESULT_POOL and SCRATCH_POOL to RESULT and
> >> SCRATCH, respectively.
> >>
> >> Because the names are too long and C is strongly typed.
> >
> > Meh.
> >
> > Or, to be more formal about it: -0.
>
> Maybe I'm just getting crotchety in my old age, but I'm also -0 on
> this proposal.

Me too. Let's keep the names as they are.

Changing this convention would also introduce naming inconsistencies
we'd have to fix up at some point, for no real gain whatsoever.
Received on 2011-09-13 19:32:26 CEST