On Wed, 6 Apr 2005, 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.
> >>>
> >>> Yes, I think that's exactly the difference between the two error codes.
> >>
> >> Thanks for clarifying it to me. I'll stop pushing for a clarification in
> >> the source code if no-one else is bothered by it.
> >
> > 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.
>
I like this change. +1.
Thanks, Julian for caring about these details,
//Peter
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Thu Apr 7 11:10:15 2005