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

[Karl Fogel <kf...@galois.collab.net>]

Greg Stein <gs...@lyra.org> writes:
> Another way to have fixed WC is to dup() first, then use the side-effect
> versions of the functions. svn_string is great for the side-effect type of
> functions because it can grow/shrink/realloc as needed for the operation.
> 
> I'd recommend one of two things:
> 
> 1) restore the side-effect style and dup first
> 2) create a second set of functions that use side-effects (maybe suffix
>    these with _se?)
> 
> >From a memory efficiency standpoint, the side-effect on an svn_string is
> "best".

Yeah, that's true..

Must run, but will sleep on it and Do The Right Thing tomorrow.