On 27.04.2013 17:01, Daniel Shahaf wrote: > Daniel Shahaf wrote on Sat, Apr 27, 2013 at 17:46:42 +0300: >> Bert Huijben wrote on Sat, Apr 27, 2013 at 16:43:29 +0200: >>> (Another option would be to start using our tristate enum for this case... >>> But we made this svnserve<->libsvn_ra_svn api public following our old >>> inter-library rules) >> +1 >> >> And svn_ra_svn__parse_number is now a private API, the public >> counterpart is deprecated (hat tip to stefan2 for the legwork). > > [[[ > Allow ra_svn protocol parsing to use svn_tristate_t instead of > a non-robust combination of TRUE, FALSE, and a custom macro.
+1 to this patch. Sorry for the mess earlier; all my digging through the docs didn't give me the impression that my change was worse than the original. -- Brane -- Branko Čibej Director of Subversion | WANdisco | www.wandisco.com