[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: svn commit: r1051164 - /subversion/trunk/subversion/libsvn_ra/util.c

From: C. Michael Pilato <cmpilato_at_collab.net>
Date: Mon, 20 Dec 2010 15:04:18 -0500

On 12/20/2010 02:45 PM, Philip Martin wrote:
> cmpilato_at_apache.org writes:
>
>> Author: cmpilato
>> Date: Mon Dec 20 16:07:08 2010
>> New Revision: 1051164
>>
>> URL: http://svn.apache.org/viewvc?rev=1051164&view=rev
>> Log:
>> Another follow-up to r1050216.
>>
>> * subversion/libsvn_ra/util.c
>> (svn_ra__release_operational_lock): Switch the sense of the boolean
>> return from svn_string_compare(). I forgot that it returns TRUE for
>> "matches", the opposite of the previous strcmp() code that was in place.
>
> The original code did "strcmp() == 0" so "matches" was TRUE there as
> well.

Doh! That's what I get for reading the code too quickly. (Thanks for the
review, Philip.)

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

Received on 2010-12-20 21:04:51 CET

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.