diff options
| author | Roland McGrath <roland@hack.frob.com> | 2014-10-20 14:13:14 -0700 |
|---|---|---|
| committer | Roland McGrath <roland@hack.frob.com> | 2014-10-20 14:13:14 -0700 |
| commit | ab49e7630f88ba9cc165dd2d855938c14ae4e158 (patch) | |
| tree | 1a6858662a25f66fb13a1533c75e854754a1ce3c | |
| parent | f50277c19df0937ea9691ab7e7c642ecd3ed3d94 (diff) | |
| download | glibc-ab49e7630f88ba9cc165dd2d855938c14ae4e158.tar.gz | |
Make internal lock-init macros return void.
| -rw-r--r-- | ChangeLog | 9 | ||||
| -rw-r--r-- | sysdeps/nptl/bits/libc-lock.h | 2 | ||||
| -rw-r--r-- | sysdeps/nptl/bits/libc-lockP.h | 8 | ||||
| -rw-r--r-- | sysdeps/nptl/bits/stdio-lock.h | 2 |
4 files changed, 15 insertions, 6 deletions
@@ -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] diff --git a/sysdeps/nptl/bits/libc-lock.h b/sysdeps/nptl/bits/libc-lock.h index 532cf2c701..66e5b93443 100644 --- a/sysdeps/nptl/bits/libc-lock.h +++ b/sysdeps/nptl/bits/libc-lock.h @@ -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 { \ diff --git a/sysdeps/nptl/bits/libc-lockP.h b/sysdeps/nptl/bits/libc-lockP.h index ec202713d6..5351a51b11 100644 --- a/sysdeps/nptl/bits/libc-lockP.h +++ b/sysdeps/nptl/bits/libc-lockP.h @@ -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) diff --git a/sysdeps/nptl/bits/stdio-lock.h b/sysdeps/nptl/bits/stdio-lock.h index e2678cc918..269df5e55e 100644 --- a/sysdeps/nptl/bits/stdio-lock.h +++ b/sysdeps/nptl/bits/stdio-lock.h @@ -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) |