<br><tt><font size=2>"Daniel P. Berrange" <berrange@redhat.com>
wrote on 03/17/2010 07:48:46 AM:<br>
Berrange"</font></tt>
<br><tt><font size=2><br>
> >  void virMutexDestroy(virMutexPtr m)<br>
> >  {<br>
> >      pthread_mutex_destroy(&m->lock);<br>
> <br>
> Minor point, I think it would be better to call it virMutexInitRecursive()<br>
> so we keep the standard virMutex prefix for naming convention<br>
> </font></tt>
<br>
<br><tt><font size=2>Ok. Done.</font></tt>
<br><tt><font size=2><br>
> <br>
> > Index: libvirt-acl/src/util/threads-pthread.h<br>
> > ===================================================================<br>
> > --- libvirt-acl.orig/src/util/threads-pthread.h<br>
> > +++ libvirt-acl/src/util/threads-pthread.h<br>
> > @@ -24,6 +24,7 @@<br>
> >  #include <pthread.h><br>
> >  <br>
> >  struct virMutex {<br>
> > +    pthread_mutexattr_t attr;<br>
> >      pthread_mutex_t lock;<br>
> >  };<br>
> <br>
> I don't think this is neccessary - the attributes are only used<br>
> at time of pthread_mutex_init() call, so don't need to be kept<br>
> around after that. Thus the 'pthread_mutexattr_t attr' could just<br>
> be a local variable in virRecursiveMutexInit().</font></tt>
<br>
<br><tt><font size=2>Ok.</font></tt>
<br>
<br><tt><font size=2>Thanks for reviewing.</font></tt>
<br>
<br><tt><font size=2> Regards,</font></tt>
<br><tt><font size=2>    Stefan</font></tt>
<br>