Re: [PATCH] add pool pointer to svn_string_t

I actually still prefer to use svn_string_t even when carrying plain
old c-string data, because:

Subversion strings provide that one extra level of indirection -- by
wrapping the data pointer, we get resizes that work in the happy way.
Also, now strings store their pools, which may help readability and
prevent allocation-lifetime bugs.

Were we to use (char *) instead, and use the apr string functions
everywhere, then things like svn_path_*() wouldn't be able to
side-effect the string's data. So we'd probably end up with
return-by-reference arguments where we currently don't need them, or
else values would have to get passed up call chains in order to
satisfy someone's original assignment statement.

Even post-M1, I'd like to keep using svn_string_t as we have been,
unless a majority of developers feel strongly about switching.

-K
Received on Sat Oct 21 14:36:11 2006