[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

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

From: <kfogel_at_collab.net>
Date: 2005-05-09 18:41:34 CEST

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.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Mon May 9 19:27:30 2005

This is an archived mail posted to the Subversion Dev mailing list.