Re: CVS update: subversion/subversion/include svn_error.h

cmpilato@tigris.org writes:

> (svn_pool_create, svn_pool_clear, svn_pool_destroy): Wrapped these
> prototypes in macro checks.

To follow up on my own commit, here.

Yesterday, we had svn_pool_create and svn_pool_clear functions, and a
bug that we are affectionately referring to as "Malloc Minnesota"
which seems to be what happens over the course of most svn binary
executions, due mostly to a combination of sloppy pool usage and a
filesystem storage inefficiency (which, by the way, has NOTHING to do
with the fact that we are using Berkeley as opposed to some flat-file
format).

I needed a way to debug our pool usage, and Jim suggested the changes
you saw in my last commit.

So, now we have an svn_pool_destroy function (which rocks, because the
sheer inconsistency in the naming of the basic pool management triplet
was rather frustrating to me as a programmer), we have some really
useful debug information *when you ask for it*, and ... we still have
the Malloc Minnesota bug--for now. I'm workin' on it.
Received on Sat Oct 21 14:36:28 2006