Fix 2008-08-03 change.

diff --git a/ChangeLog b/ChangeLog
index d60d1e2..5f017b5 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,12 @@
 2008-08-18  Bruno Haible  <bruno@clisp.org>
 
+       * lib/glthread/lock.h [USE_SOLARIS_THREADS]: Fix
+       glthread_recursive_lock_* macros.
+       * lib/glthread/lock.c (glthread_recursive_lock_destroy_multithreaded):
+       Fix syntax error.
+
+2008-08-18  Bruno Haible  <bruno@clisp.org>
+
        * lib/glthread/thread.c: Avoid forcing a context switch right after
        thread creation.
 

diff --git a/lib/glthread/lock.c b/lib/glthread/lock.c
index 2ad0e06..487d23d 100644 (file)
--- a/lib/glthread/lock.c
+++ b/lib/glthread/lock.c
@@ -445,7 +445,7 @@ glthread_recursive_lock_destroy_multithreaded (gl_recursive_lock_t *lock)
 {
   if (lock->owner != (pthread_t) 0)
     return EBUSY;
-  return (pthread_mutex_destroy (&lock->mutex);
+  return pthread_mutex_destroy (&lock->mutex);
 }
 
 # endif

diff --git a/lib/glthread/lock.h b/lib/glthread/lock.h
index fd74941..391165a 100644 (file)
--- a/lib/glthread/lock.h
+++ b/lib/glthread/lock.h
@@ -577,33 +577,13 @@ typedef struct
 # define gl_recursive_lock_initializer \
     { DEFAULTMUTEX, (thread_t) 0, 0 }
 # define glthread_recursive_lock_init(LOCK) \
-    do                                          \
-      {                                         \
-        if (thread_in_use ())                   \
-          glthread_recursive_lock_init_multithreaded (LOCK); \
-      }                                         \
-    while (0)
+    (thread_in_use () ? glthread_recursive_lock_init_multithreaded (LOCK) : 0)
 # define glthread_recursive_lock_lock(LOCK) \
-    do                                          \
-      {                                         \
-        if (thread_in_use ())                   \
-          glthread_recursive_lock_lock_multithreaded (LOCK); \
-      }                                         \
-    while (0)
+    (thread_in_use () ? glthread_recursive_lock_lock_multithreaded (LOCK) : 0)
 # define glthread_recursive_lock_unlock(LOCK) \
-    do                                            \
-      {                                           \
-        if (thread_in_use ())                     \
-          glthread_recursive_lock_unlock_multithreaded (LOCK); \
-      }                                           \
-    while (0)
+    (thread_in_use () ? glthread_recursive_lock_unlock_multithreaded (LOCK) : 0)
 # define glthread_recursive_lock_destroy(LOCK) \
-    do                                             \
-      {                                            \
-        if (thread_in_use ())                      \
-          glthread_recursive_lock_destroy_multithreaded (LOCK); \
-      }                                            \
-    while (0)
+    (thread_in_use () ? glthread_recursive_lock_destroy_multithreaded (LOCK) : 0)
 extern int glthread_recursive_lock_init_multithreaded (gl_recursive_lock_t *lock);
 extern int glthread_recursive_lock_lock_multithreaded (gl_recursive_lock_t *lock);
 extern int glthread_recursive_lock_unlock_multithreaded (gl_recursive_lock_t *lock);