diff options
| author | Roland McGrath <roland@hack.frob.com> | 2014-02-07 15:26:28 -0800 |
|---|---|---|
| committer | Roland McGrath <roland@hack.frob.com> | 2014-02-07 15:26:28 -0800 |
| commit | 32749f6cd23f65a2df1bafbf9b69749bdded4b10 (patch) | |
| tree | 286f6a962fd5c4b8e539cc59d4f6b2f8b0b1edc2 | |
| parent | d5b396c1c89ed3026fc89bfcdd72b14d59972e45 (diff) | |
| download | glibc-32749f6cd23f65a2df1bafbf9b69749bdded4b10.zip glibc-32749f6cd23f65a2df1bafbf9b69749bdded4b10.tar.gz glibc-32749f6cd23f65a2df1bafbf9b69749bdded4b10.tar.bz2 | |
Avoid comma operator warnings.
| -rw-r--r-- | ChangeLog | 9 | ||||
| -rw-r--r-- | bits/sigset.h | 10 | ||||
| -rw-r--r-- | include/signal.h | 3 | ||||
| -rw-r--r-- | sysdeps/generic/malloc-machine.h | 2 |
4 files changed, 18 insertions, 6 deletions
@@ -1,3 +1,12 @@ +2014-02-07 Roland McGrath <roland@hack.frob.com> + + * bits/sigset.h (__sigemptyset): Use a statement expression rather + than the comma operator, to avoid "rhs of comma has no effect" + compiler warnings. + (__sigfillset, __sigandset, __sigorset): Likewise. + * include/signal.h (__sigemptyset): Likewise. + * sysdeps/generic/malloc-machine.h (mutex_lock): Likewise. + 2014-02-07 Allan McRae <allan@archlinux.org> * version.h (RELEASE): Set to "development". diff --git a/bits/sigset.h b/bits/sigset.h index 23a6c1f..e5fac54 100644 --- a/bits/sigset.h +++ b/bits/sigset.h @@ -44,15 +44,17 @@ typedef unsigned long int __sigset_t; overflow if `sigset_t' is wider than `int'. */ #define __sigmask(sig) (((__sigset_t) 1) << ((sig) - 1)) -#define __sigemptyset(set) ((*(set) = (__sigset_t) 0), 0) -#define __sigfillset(set) ((*(set) = ~(__sigset_t) 0), 0) +#define __sigemptyset(set) \ + (__extension__ ({ *(set) = (__sigset_t) 0; 0; }) +#define __sigfillset(set) \ + (__extension__ ({ *(set) = ~(__sigset_t) 0; 0; })) #ifdef _GNU_SOURCE # define __sigisemptyset(set) (*(set) == (__sigset_t) 0) # define __sigandset(dest, left, right) \ - ((*(dest) = (*(left) & *(right))), 0) + (__extension__ ({ *(dest) = *(left) & *(right); 0; })) # define __sigorset(dest, left, right) \ - ((*(dest) = (*(left) | *(right))), 0) + (__extension__ ({ *(dest) = *(left) | *(right); 0; })) #endif /* These functions needn't check for a bogus signal number -- error diff --git a/include/signal.h b/include/signal.h index a019395..16a7a42 100644 --- a/include/signal.h +++ b/include/signal.h @@ -55,7 +55,8 @@ extern int __xpg_sigpause (int sig); /* Simplified sigemptyset() implementation without the parameter checking. */ #undef __sigemptyset -#define __sigemptyset(ss) (__builtin_memset (ss, '\0', sizeof (sigset_t)), 0) +#define __sigemptyset(ss) \ + ({ __builtin_memset (ss, '\0', sizeof (sigset_t)); 0; }) /* Allocate real-time signal with highest/lowest available priority. */ diff --git a/sysdeps/generic/malloc-machine.h b/sysdeps/generic/malloc-machine.h index df7adeb..a1b7655 100644 --- a/sysdeps/generic/malloc-machine.h +++ b/sysdeps/generic/malloc-machine.h @@ -35,7 +35,7 @@ typedef int mutex_t; # define mutex_init(m) (*(m) = 0) -# define mutex_lock(m) ((*(m) = 1), 0) +# define mutex_lock(m) ({ *(m) = 1; 0; }) # define mutex_trylock(m) (*(m) ? 1 : ((*(m) = 1), 0)) # define mutex_unlock(m) (*(m) = 0) # define MUTEX_INITIALIZER (0) |