aboutsummaryrefslogtreecommitdiff
path: root/libdecnumber
diff options
context:
space:
mode:
authorMarek Polacek <polacek@redhat.com>2014-01-23 19:04:29 +0000
committerMarek Polacek <mpolacek@gcc.gnu.org>2014-01-23 19:04:29 +0000
commit789eadcd54ffe62037461a4a22c2737ad9cbd01e (patch)
tree98eacc6f827b22b6ab595fbe3aaa8fffd51e38b5 /libdecnumber
parent20f18c3ce14154eb477226aa48a12ebd32f9ead4 (diff)
downloadgcc-789eadcd54ffe62037461a4a22c2737ad9cbd01e.zip
gcc-789eadcd54ffe62037461a4a22c2737ad9cbd01e.tar.gz
gcc-789eadcd54ffe62037461a4a22c2737ad9cbd01e.tar.bz2
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')
-rw-r--r--libdecnumber/ChangeLog5
-rw-r--r--libdecnumber/decNumberLocal.h7
2 files changed, 8 insertions, 4 deletions
diff --git a/libdecnumber/ChangeLog b/libdecnumber/ChangeLog
index 36a6003..14df41c 100644
--- a/libdecnumber/ChangeLog
+++ b/libdecnumber/ChangeLog
@@ -1,3 +1,8 @@
+2014-01-23 Marek Polacek <polacek@redhat.com>
+
+ PR c/59871
+ * decNumberLocal.h (UBFROMUS, UBFROMUI): Remove last argument.
+
2014-01-02 Richard Sandiford <rdsandiford@googlemail.com>
Update copyright years
diff --git a/libdecnumber/decNumberLocal.h b/libdecnumber/decNumberLocal.h
index 94e7f7f..4936231 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] */