Re: svn commit: r19282 - in branches/svnserve-ssl/subversion: include libsvn_ra_svn libsvn_subr svnserve

On Sat, Apr 08, 2006 at 10:33:07PM -0700, Garrett Rooney wrote:
> On 4/8/06, mark benedetto king <mbk@lowlatency.com> wrote:
> > On Sat, Apr 08, 2006 at 07:13:12PM -0700, Garrett Rooney wrote:
> > > On 4/8/06, mbk@tigris.org <mbk@tigris.org> wrote:
> > >
> > > > /**
> > > > + *
> > > > + * Initialize the OpenSSL library.
> > > > + *
> > > > + * @since New in 1.4.
> > > > + */
> > > > +svn_error_t *svn_ra_svn__ssl_initialize(apr_pool_t *pool);
> > >
> > > Is there a less public place we can declare this? I mean since it's a
> > > private interface there seems to be little reason to put it in a
> > > public header file.
> > >
> >
> > None are jumping out at me.
>
> A private header inside libsvn_ra_svn? We can include that from
> within svnserve, as well as inside libsvn_ra_svn itself, just like we
> do for the internal libsvn_fs headers...
>

I didn't realize that precedent had been set.

How should we handle the other "only for svnserve" functions in
svn_ra_svn.h?

--ben

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Fri Apr 14 12:59:58 2006