On Apr 22, 2012 2:09 PM, "Stefan Fuhrmann" <eq...@web.de> wrote: > > Am 16.04.2012 19:45, schrieb Greg Stein: >>... >> I also find the need for the has_sufficient_privileges() function to be problematic. It seems to indicate you have not encapsulated this stuff properly. > > Indeed, one might see this as an encapsulation issue. With APR.
No doubt. But that doesn't mean you propagate the problem into SVN. I would suggest a helper function in libsvn_subr that does this platform-specific stuff so that the app (or test, in this case) can call that helper. That encapsulates stuff away from the app, even though our library may still need to compensate for APR problems. Cheers, -g
