After verifying that taking the lock after the bookkeeping (instead of before) does not break the testsuite, I committed this comment.
> --- lib/glthread/cond.c.orig 2008-10-12 22:05:50.000000000 +0200 > +++ lib/glthread/cond.c 2008-10-12 21:30:47.000000000 +0200 > @@ -159,7 +159,8 @@ > break; > } > > - /* Take the lock again. */ > + /* Take the lock again. It does not matter whether this is done > + before or after the bookkeeping. */ > err = glthread_lock_lock (lock); > > /* Do the bookkeeping. */ > @@ -285,7 +286,8 @@ > break; > } > > - /* Take the lock again. */ > + /* Take the lock again. It does not matter whether this is done > + before or after the bookkeeping. */ > err = glthread_lock_lock (lock); > > /* Do the bookkeeping. */ > >
