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

Re: [PATCH] Re: svn commit: r1476563 - /subversion/trunk/subversion/svnserve/serve.c

From: Branko Čibej <brane_at_wandisco.com>
Date: Sat, 27 Apr 2013 17:45:32 +0200

On 27.04.2013 17:01, Daniel Shahaf wrote:
> Daniel Shahaf wrote on Sat, Apr 27, 2013 at 17:46:42 +0300:
>> Bert Huijben wrote on Sat, Apr 27, 2013 at 16:43:29 +0200:
>>> (Another option would be to start using our tristate enum for this case...
>>> But we made this svnserve<->libsvn_ra_svn api public following our old
>>> inter-library rules)
>> +1
>>
>> And svn_ra_svn__parse_number is now a private API, the public
>> counterpart is deprecated (hat tip to stefan2 for the legwork).
>
> [[[
> Allow ra_svn protocol parsing to use svn_tristate_t instead of
> a non-robust combination of TRUE, FALSE, and a custom macro.

+1 to this patch. Sorry for the mess earlier; all my digging through the
docs didn't give me the impression that my change was worse than the
original.

-- Brane

-- 
Branko Čibej
Director of Subversion | WANdisco | www.wandisco.com
Received on 2013-04-27 17:46:08 CEST

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