Re: svn commit: r1326688 - /subversion/trunk/subversion/tests/libsvn_subr/named_atomic-test.c

On Apr 22, 2012 2:09 PM, "Stefan Fuhrmann" <eqfox_at_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
Received on 2012-04-22 20:30:26 CEST