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

"Garrett Rooney" <rooneg@electricjellyfish.net> writes:

> 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?

Ahh, the 4096 I queried in the original patch :) How about getting rid
of SVN_RA_SVN__DEFAULT_SECPROPS and using

void svn_ra_svn__default_secprops(sasl_security_properties_t *secprops);

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