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

Re: Patch: add svn_client_get_repos_path

From: Peter N. Lundblad <peter_at_famlundblad.se>
Date: 2005-04-01 09:15:25 CEST

On Thu, 31 Mar 2005, Barry Scott wrote:

> On Mar 31, 2005, at 12:40, Peter N. Lundblad wrote:
>
> > instead of new special-case functions, maybe we should export some
> > general
> > purpose client function to open an RA session, such as
> > svn_client__ra_session_from_path?
>
> I used this function to test out the code fragment I posted. Making
> this function
> a supported interface would make me happy.
>
> Peter would you be willing to turn svn_client__ra_session_from_path
> into a
> supported svn_client_ra_session_from_path?
>
I'll apply a patch from you, if no one else objects. Just rename the
function (removing one _), fix the docstring to be doxygen, update the
call sites and all should be fine. I'll do it for 1.2 if it gets ready
before the branch is created.

> I hope Max that you understand the reason for this being in either in
> svn_client or
> implementing Peters suggestion to make svn_client__ra_session_from_path
> a supported API.
> I was surprised by the suggestion that this obvious improvement needed
> explaining.
>
I think every addition to the public API needs an explanation. If we add
an API, we have to live with it until 2.0.

Regards,
//Peter

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Fri Apr 1 09:11:44 2005

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

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.