Re: svn commit: r1060937 - in /subversion/branches/uris-as-urls/subversion: libsvn_fs_base/bdb/changes-table.c libsvn_fs_base/bdb/locks-table.c libsvn_fs_fs/lock.c

On 01/19/2011 04:21 PM, Ivan Zhakov wrote:
> On Wed, Jan 19, 2011 at 22:52, <cmpilato_at_apache.org> wrote:

[...]

>> Modified: subversion/branches/uris-as-urls/subversion/libsvn_fs_base/bdb/locks-table.c
>> URL: http://svn.apache.org/viewvc/subversion/branches/uris-as-urls/subversion/libsvn_fs_base/bdb/locks-table.c?rev=1060937&r1=1060936&r2=1060937&view=diff
>> ==============================================================================
>> --- subversion/branches/uris-as-urls/subversion/libsvn_fs_base/bdb/locks-table.c (original)
>> +++ subversion/branches/uris-as-urls/subversion/libsvn_fs_base/bdb/locks-table.c Wed Jan 19 19:52:24 2011
>> @@ -249,8 +249,9 @@ svn_fs_bdb__locks_get(svn_fs_t *fs,
>>
>> /* As long as the prefix of the returned KEY matches LOOKUP_PATH we
>> know it is either LOOKUP_PATH or a decendant thereof. */
>> - if (strcmp(path, "/") != 0)
>> + if (svn_fspath__is_root(path, strlen(path)))
>
> ^^^^ I think you missed "!" there. It should be "if
> (!svn_fspath__is_root(path, strlen(path)))".

Yep! Thanks, Ivan. (And please keep reviewing commits!)

-- 
C. Michael Pilato <cmpilato_at_collab.net>
CollabNet   <>   www.collab.net   <>   Distributed Development On Demand