Re: svn commit: r1767194 - /subversion/trunk/subversion/libsvn_ra_svn/marshal.c

On 31.10.2016 01:19, Daniel Shahaf wrote:
> stefan2_at_apache.org wrote on Sun, Oct 30, 2016 at 22:30:19 -0000:
>> Author: stefan2
>> Date: Sun Oct 30 22:30:19 2016
>> New Revision: 1767194
>>
>> URL: http://svn.apache.org/viewvc?rev=1767194&view=rev
>> Log:
>> Close a gap in the ra_svn protocol engine:
>> Boolean elements in conditional tuples were not supported but would also not
>> occur in the protocol definition. They will in the future, though.
>>
>
> The relevant private API docstrings state:
>
> * 'b' may not appear inside an optional tuple specification; use '3' instead.
>
> Please bring the API docs back in sync with the code. (Not sure whether
> it's better to revert the code change and use '3' instead in the caller,
> or to update the docs to match the new code.)

Oh! You are right.

This ties in to the protocol change in r1767197 - see there.

-- Stefan^2.
Received on 2016-11-05 11:59:06 CET