Re: svn commit: r37894 - trunk/subversion/libsvn_subr

Paul T. Burba wrote:
> Author: pburba
> Date: Fri May 29 11:59:01 2009
> New Revision: 37894

[...]

> /* Hmmm. Ideally, we would append to a native-encoding buf
> before starting iteration, then convert back to UTF-8 for
> return. But I suppose that would make the appending code
> sensitive to i18n in a way it shouldn't be... Oh well. */
> - /* ### could alloc in scratch, then copy to result in baton... */
> - SVN_ERR(cstring_from_utf8(&unique_name_apr, unique_name, result_pool));
> + SVN_ERR(cstring_from_utf8(&unique_name_apr, unique_name, scratch_pool));

Do we also need need to dup unique_name_apr to result_pool when we set
baton->name ?

-- 
C. Michael Pilato <cmpilato_at_collab.net>
CollabNet   <>   www.collab.net   <>   Distributed Development On Demand
------------------------------------------------------
http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=2356919