Re: [PATCH] add pool pointer to svn_string_t

I support this change, but it might be nice to hear the opion of one
of the original Subversion people for such a basic change.

> + /* pool from which the string was originally allocated -- used only
> + for growing the string, and is NOT destroyed when the string is
> + destroyed */

You can't destroy a Subversion string, so this comment doesn't quite
make sense. I'm not sure what alternative wording to use; perhaps
"and is NOT necessarily specific to the string".

> This also removes the feature that svn_string_append* will allocate
> the string if they are passed the NULL.

That feature doesn't make sense anyway. (A string is allocated and
then immediately forgotten about.) I think someone wasn't paying
attention when they put that in.
Received on Sat Oct 21 14:36:10 2006