[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_ra_initialize

From: Philip Martin <philip_at_codematters.co.uk>
Date: 2005-04-14 01:19:27 CEST

kfogel@collab.net writes:

> "Peter N. Lundblad" <peter@famlundblad.se> writes:
>> Short summary: I'm crazy.
>>
>> Longer: I want to add an API to 1.2 that does nothing. That's the
>> svn_ra_initialize I mentioned in the thread about the fSFS fix. We will
>> need it in 1.3 to properly support calling svn_fs_initialize from the
>> client. Would it be accepted into 1.2?
>
> Maybe I need the _really_ long version...
>
> What's the purpose of putting it into 1.2, when we are allowed to add
> it in 1.3 anyway?

If svn_ra_initialize is present in 1.2 then we can require that it be
called before any other "new in 1.2" function. Most of the svn_ra_xxx
stuff happens to be new in 1.2, we deprecated the vtable approach.

I'm not sure why the patch doesn't implement svn_ra_initialize, just a
lack of time perhaps? If the --enable-dso problem is considered to be
sufficiently important to block the release then we have to implement
something, and it may as well be svn_ra_initialize.

-- 
Philip Martin
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Thu Apr 14 01:20:19 2005

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