On Mon, Apr 30, 2012 at 23:31, Hyrum K Wright <[email protected]> wrote:
> On Mon, Apr 30, 2012 at 10:02 PM, Greg Stein <[email protected]> wrote:
>>
>> On Apr 30, 2012 9:38 PM, "Hyrum K Wright" <[email protected]> wrote:
>>>...
>>> As the key of the hash, RELPATH should be duplicated into a pool with
>>> sufficient lifetime.
>>
>> Good catch! Fixed in r1332508.
>
> Thanks.  For the same reason, you probably want to
> s/apr_hash_copy/svn_prop_hash_dup/ over compat.c.  The former only
> duplicates the hash, not the keys or the contents, while the latter
> will ensure the prophash has the same lifetime as the change it
> belongs to.

Oop! You're right. Will do after I commit my work-in-progress.

Thx,
-g

Reply via email to