]> git.ipfire.org Git - thirdparty/glibc.git/commitdiff
Make internal lock-init macros return void.
authorRoland McGrath <roland@hack.frob.com>
Mon, 20 Oct 2014 21:13:14 +0000 (14:13 -0700)
committerRoland McGrath <roland@hack.frob.com>
Mon, 20 Oct 2014 21:13:14 +0000 (14:13 -0700)
ChangeLog
sysdeps/nptl/bits/libc-lock.h
sysdeps/nptl/bits/libc-lockP.h
sysdeps/nptl/bits/stdio-lock.h

index f6a7e43c329ac30e1001b5daec7679ba6cb54400..e9d9d8d7376fa0e60ed75ddaba16c98e93729a06 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,12 @@
+2014-10-20  Roland McGrath  <roland@hack.frob.com>
+
+       * sysdeps/nptl/bits/libc-lock.h
+       [_LIBC && (!NOT_IN_libc || IS_IN_libpthread)]
+       (__libc_lock_init_recursive): Return void, not 0.
+       * sysdeps/nptl/bits/libc-lockP.h (__libc_lock_init): Likewise.
+       (__libc_rwlock_init): Likewise.
+       * sysdeps/nptl/bits/stdio-lock.h (_IO_lock_init): Likewise.
+
 2014-10-20  Torvald Riegel  <triegel@redhat.com>
 
        [BZ #15215]
index 532cf2c701d3d68ea2b1c62194ef775ab764760a..66e5b9344343a12b2eca58b66bb2b1da6a024d47 100644 (file)
@@ -62,7 +62,7 @@ typedef struct __libc_lock_recursive_opaque__ __libc_lock_recursive_t;
 /* Initialize a recursive mutex.  */
 #if defined _LIBC && (!defined NOT_IN_libc || defined IS_IN_libpthread)
 # define __libc_lock_init_recursive(NAME) \
-  ((NAME) = (__libc_lock_recursive_t) _LIBC_LOCK_RECURSIVE_INITIALIZER, 0)
+  ((void) ((NAME) = (__libc_lock_recursive_t) _LIBC_LOCK_RECURSIVE_INITIALIZER))
 #else
 # define __libc_lock_init_recursive(NAME) \
   do {                                                                       \
index ec202713d639f2cb49ef6a0f44dd70d8de3e7dd6..5351a51b1100b4c89b194dec404f1f13976abf9d 100644 (file)
@@ -126,16 +126,16 @@ typedef pthread_key_t __libc_key_t;
 /* Initialize the named lock variable, leaving it in a consistent, unlocked
    state.  */
 #if !defined NOT_IN_libc || defined IS_IN_libpthread
-# define __libc_lock_init(NAME) ((NAME) = LLL_LOCK_INITIALIZER, 0)
+# define __libc_lock_init(NAME) \
+  ((void) ((NAME) = LLL_LOCK_INITIALIZER))
 #else
 # define __libc_lock_init(NAME) \
   __libc_maybe_call (__pthread_mutex_init, (&(NAME), NULL), 0)
 #endif
 #if defined SHARED && !defined NOT_IN_libc
-/* ((NAME) = (__libc_rwlock_t) PTHREAD_RWLOCK_INITIALIZER, 0) is
-   inefficient.  */
+/* ((NAME) = (__libc_rwlock_t) PTHREAD_RWLOCK_INITIALIZER) is inefficient.  */
 # define __libc_rwlock_init(NAME) \
-  (__builtin_memset (&(NAME), '\0', sizeof (NAME)), 0)
+  ((void) __builtin_memset (&(NAME), '\0', sizeof (NAME)))
 #else
 # define __libc_rwlock_init(NAME) \
   __libc_maybe_call (__pthread_rwlock_init, (&(NAME), NULL), 0)
index e2678cc918ca6443a5c0719a20b104f3bd1d5c2b..269df5e55eacab95e28c8958a69c84e1930e7567 100644 (file)
@@ -31,7 +31,7 @@ typedef struct { int lock; int cnt; void *owner; } _IO_lock_t;
 #define _IO_lock_initializer { LLL_LOCK_INITIALIZER, 0, NULL }
 
 #define _IO_lock_init(_name) \
-  ((_name) = (_IO_lock_t) _IO_lock_initializer , 0)
+  ((void) ((_name) = (_IO_lock_t) _IO_lock_initializer))
 
 #define _IO_lock_fini(_name) \
   ((void) 0)