Re: svn commit: r21554 - in trunk/subversion: include libsvn_ra_svn libsvn_repos svnserve

On 9/19/06, Daniel L. Rall <dlr@finemaltcoding.com> wrote:
> On Tue, 19 Sep 2006, Garrett Rooney wrote:
> ...
> > Ok, one catch... ra_svn_sasl.h depends on ra_svn.h for one constant
> > (SVN_RA_SVN__READBUF_SIZE). Now there's no way in hell that
> > SVN_RA_SVN__READBUF_SIZE should live in ra_svn_sasl.h, and similarly,
> > I'm not all that thrilled about ra_svn.h living in private/, since
> > it's really not used outside of libsvn_ra_svn. Any thoughts on a
> > proper home for that constant?
>
> SVN_RA_SVN__DEFAULT_SECPROPS is only used in one place:

It is actually used in two places: in libsvn_ra_svn/sasl_auth.c and in
svnserve/sasl_auth.c.

I just posted a patch that attempts to fix this problem by introducing
a new function, like Philip suggested.

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