diff options
author | Marek Polacek <polacek@redhat.com> | 2014-01-23 19:04:29 +0000 |
---|---|---|
committer | Marek Polacek <mpolacek@gcc.gnu.org> | 2014-01-23 19:04:29 +0000 |
commit | 789eadcd54ffe62037461a4a22c2737ad9cbd01e (patch) | |
tree | 98eacc6f827b22b6ab595fbe3aaa8fffd51e38b5 /libdecnumber/decNumberLocal.h | |
parent | 20f18c3ce14154eb477226aa48a12ebd32f9ead4 (diff) |
re PR c/59871 (No unused value warning for comma expression)
PR c/59871
c/
* c-typeck.c (build_compound_expr): Warn even for right-hand operand
of a comma expression.
(emit_side_effect_warnings): Likewise.
libdecnumber/
* decNumberLocal.h (UBFROMUS, UBFROMUI): Remove last argument.
testsuite/
* gcc.dg/20020220-2.c: Adjust dg-warning message.
* gcc.dg/pr59871.c: New test.
From-SVN: r207002
Diffstat (limited to 'libdecnumber/decNumberLocal.h')
-rw-r--r-- | libdecnumber/decNumberLocal.h | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/libdecnumber/decNumberLocal.h b/libdecnumber/decNumberLocal.h index 94e7f7f9b1f..4936231f2a2 100644 --- a/libdecnumber/decNumberLocal.h +++ b/libdecnumber/decNumberLocal.h @@ -153,10 +153,9 @@ see the files COPYING3 and COPYING.RUNTIME respectively. If not, see #define UBTOUI(b) (memcpy((void *)&uiwork, b, 4), uiwork) /* Store a uInt, etc., into bytes starting at a char* or uByte*. */ - /* Returns i, evaluated, for convenience; has to use uiwork because */ - /* i may be an expression. */ - #define UBFROMUS(b, i) (uswork=(i), memcpy(b, (void *)&uswork, 2), uswork) - #define UBFROMUI(b, i) (uiwork=(i), memcpy(b, (void *)&uiwork, 4), uiwork) + /* Has to use uiwork because i may be an expression. */ + #define UBFROMUS(b, i) (uswork=(i), memcpy(b, (void *)&uswork, 2)) + #define UBFROMUI(b, i) (uiwork=(i), memcpy(b, (void *)&uiwork, 4)) /* X10 and X100 -- multiply integer i by 10 or 100 */ /* [shifts are usually faster than multiply; could be conditional] */ |