On Thu, Oct 13, 2011 at 04:58:56PM -0000, stef...@apache.org wrote: > Author: stefan2 > Revision: 1182979 > Modified property: svn:log > > Modified: svn:log at Thu Oct 13 16:58:56 2011 > ------------------------------------------------------------------------------ > --- svn:log (original) > +++ svn:log Thu Oct 13 16:58:56 2011 > @@ -1,2 +1,6 @@ > +Remove a #if APR_HAS_THREADS statement. They have been one reason for > +introducing svn_mutex__t in the first place.
So you're saying that svn_mutex__t is intended to provide an abstraction layer which hides APR_HAS_THREADS? If so, why does svn_mutex__init() take a boolean parameter 'enable_mutex', which essentially specifies whether APR supports threads? Shouldn't the function figure that out by itself?
