Brian W. Fitzpatrick wrote:
> On Apr 6, 2005, at 5:25 PM, Julian Foad wrote:
>> Max Bowsher wrote:
>>> Julian Foad wrote:
>>>> Ben Collins-Sussman wrote:
>>>>> On Apr 4, 2005, at 2:36 PM, Julian Foad wrote:
>>>>>> Aha! I have just seen that "SVN_ERR_FS_PATH_LOCKED" is tested by the
>>>>>> macro "SVN_ERR_IS_LOCK_ERROR", and therefore perhaps is intended only
>>>>>> to mean, "You can't lock this path because it is already locked". If
>>>>>> that's the case, may we rename it to
>>>>>> "SVN_ERR_FS_PATH_ALREADY_LOCKED"?
>>>>>> Or at least document it as such.
[...]
>>> Actually, now it's been worked out what the issue is, I think this
>>> very much *should* be clarified.
>>> Preferably by changing the name to _ALREADY_LOCKED as suggested above.
>>
>> Here's a patch to do so. Note that this can only be done if it is
>> ported into v1.2 before v1.2 is released, otherwise it would be an
>> incompatible API change.
>
> Much clearer. +1
Right, that's now three other agreements. Committed in r14011 and proposed for
back-port into v1.2.0.
- Julian
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Thu Apr 7 17:53:58 2005