summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorRoland McGrath <roland@hack.frob.com>2016-01-20 13:57:14 -0800
committerRoland McGrath <roland@hack.frob.com>2016-01-20 13:57:14 -0800
commita3140836c82679f85b780e936ae0c0179997952e (patch)
treea46a76c1f1b83c0657509f2ce2ebcabfba21961d
parentb274130206cb9fce1d7b026386a910428cbdb45c (diff)
NaCl: Fix unused variable errors in lowlevellock-futex.h macros.
-rw-r--r--ChangeLog6
-rw-r--r--sysdeps/nacl/lowlevellock-futex.h7
2 files changed, 11 insertions, 2 deletions
diff --git a/ChangeLog b/ChangeLog
index ac8eb40ceb..f1c6ac5cd0 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,9 @@
+2016-01-20 Roland McGrath <roland@hack.frob.com>
+
+ * sysdeps/nacl/lowlevellock-futex.h
+ (lll_futex_wait, lll_futex_timed_wait, lll_futex_wake):
+ Always evaluate PRIVATE argument.
+
2016-01-20 Paul Pluzhnikov <ppluzhnikov@google.com>
[BZ #19490]
diff --git a/sysdeps/nacl/lowlevellock-futex.h b/sysdeps/nacl/lowlevellock-futex.h
index 5f7d45950b..5382f685ff 100644
--- a/sysdeps/nacl/lowlevellock-futex.h
+++ b/sysdeps/nacl/lowlevellock-futex.h
@@ -35,7 +35,8 @@
/* Wait while *FUTEXP == VAL for an lll_futex_wake call on FUTEXP. */
#define lll_futex_wait(futexp, val, private) \
- (- __nacl_irt_futex.futex_wait_abs ((volatile int *) (futexp), val, NULL))
+ ((void) (private), \
+ - __nacl_irt_futex.futex_wait_abs ((volatile int *) (futexp), val, NULL))
/* Wait until a lll_futex_wake call on FUTEXP, or TIMEOUT elapses. */
#define lll_futex_timed_wait(futexp, val, timeout, private) \
@@ -60,13 +61,15 @@
if (_err == 0) \
_err = __nacl_irt_futex.futex_wait_abs \
((volatile int *) (futexp), val, _to); \
- -_err; \
+ (void) (private); \
+ -_err; \
})
/* Wake up up to NR waiters on FUTEXP. */
#define lll_futex_wake(futexp, nr, private) \
({ \
int _woken; \
+ (void) (private); \
- __nacl_irt_futex.futex_wake ((volatile int *) (futexp), nr, &_woken); \
})