Re: svn commit: rev 4012 - trunk/subversion/libsvn_client

Justin Erenkrantz <jerenkrantz@apache.org> writes:

> --On Thursday, December 5, 2002 12:13 PM -0600 cmpilato@tigris.org
> wrote:
>
> > Author: cmpilato
> > Date: 2002-12-05 12:13:33 -0600 (Thu, 05 Dec 2002)
> > New Revision: 4012
> >
> > Modified:
> > trunk/subversion/libsvn_client/commit_util.c
> > Log:
> > * subversion/libsvn_client/commit_util.c
> > (harvest_committables): Cleanup some wacky pool usage in here, and
> > remove some unnecessary strdup's. Also, add a buncha comments.
>
> That wacky pool usage was on purpose.
>
> harvest_committables itself is executed in a loop many times (think of
> the recursive case).

And the pool handing to it in those looping cases is itself a subpool
that gets cleared per-iteration. The effects of my change were to go
from:

   while ()
      begin harvest_committables
        make subpool
        use subpool
        destroy subpool
      end harvest_committables

to:

   while ()
      make subpool
      begin harvest_committables
        use subpool
      end harvest_committables
      destroy subpool

which is how we typically expect pool usage in Subversion to work.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Thu Dec 5 20:05:10 2002