Re: adm_access confusion

Greg Stein <gstein@lyra.org> writes:

> On Mon, Mar 31, 2003 at 03:49:32PM -0600, Ben Collins-Sussman wrote:
> >...
> > * svn_client.h (svn_client_uuid_from_path, svn_client_uuid_from_url):
> > declare. new utility functions for getting uuids from either
> > entries files or via RA->get_uuid().
> >
> > (svn_client_add): take an optional_adm_access argument.
>
> Bleck. I don't think we should be exposing that in the client interface.
> Heck, even the comment for svn_client_copy() says the argument is temporary.

Agreed.

> I'm not sure why we have it in svn_client_delete().

I took a shortcut while doing the access baton conversion, merge calls
svn_client_delete and it has already locked the working copy.

-- 
Philip Martin
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org