You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@subversion.apache.org by Stefan Fuhrmann <st...@apache.org> on 2016/11/05 10:48:11 UTC

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

On 31.10.2016 01:19, Daniel Shahaf wrote:
> stefan2@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.