Re: svn commit: r14643 - in trunk/subversion: include libsvn_client libsvn_ra_dav libsvn_repos libsvn_wc

kfogel@collab.net wrote:
> Greg Hudson <ghudson@MIT.EDU> writes:
>>On Mon, 2005-05-09 at 01:07 -0500, kfogel@tigris.org wrote:
>>>+ if (pool == NULL)
>>>+ {
>>>+ pool = svn_pool_create (NULL);
>>>+ own_pool = TRUE;
>>>+ }
>>
>>A better way: create an internal function which accepts the buffer on
>>the command line. Make the old API allocate the buffer on the stack,
>>and the new one allocate it within the provided pool.
>
> Presumably s/on the command line/as a parameter/.
>
> But r14647 is an even better way IMHO, and it's now in STATUS.

But r14647 removes all mention of argument "pool" from the API doc. Has it
been voted through to branches/1.2.x already? Tut, tut :-)

- Julian

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Tue May 10 14:43:27 2005