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

Re: ra_dav->unlock()

From: Peter N. Lundblad <peter_at_famlundblad.se>
Date: 2005-01-07 16:23:03 CET

On Fri, 7 Jan 2005, C. Michael Pilato wrote:

> Ben Collins-Sussman <sussman@collab.net> writes:
>
> > On Jan 7, 2005, at 8:56 AM, C. Michael Pilato wrote:
> >
> > > Ben Collins-Sussman <sussman@collab.net> writes:
> > >
> > >> But as a matter of completeness, it seems odd for our fs locking API
> > >> to return tokens that "represent locks", and yet not have an API to
> > >> lookup by token.
> > >
> > > There is precedent. The FS API can return node IDs that "represent
> > > node revisions", but you can't query any other information using those
> > > IDs. The API exists purely so you can compare IDs.
> > >
> > >
> >
> > So... is that a vote to remove svn_fs_get_lock_from_token()
> > altogether?
>
> If it isn't needed, then it just becomes baggage we have to drag
> around. I'll let those with the knowledge say if it's needed or not,
> but if not, then yes, my vote is for removal.
>
ra_svn doesn't need it. I have no strong opinion, but I lean towards
agreeing to remove it if we don't need it for DAV.

Regards,
//Peter

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Fri Jan 7 16:37:45 2005

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