Greg Stein wrote:
>
>
> On Apr 22, 2012 2:09 PM, "Stefan Fuhrmann" <eqfox_at_web.de
> <mailto: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.
>
Addressed by r1331760.
Thanks for the review!
-- Stefan^2.
Received on 2012-04-28 16:01:59 CEST