diff options
| author | mpolacek <mpolacek@138bc75d-0d04-0410-961f-82ee72b054a4> | 2014-01-23 19:04:29 +0000 |
|---|---|---|
| committer | Tom Tromey <tromey@redhat.com> | 2014-05-08 10:18:28 -0600 |
| commit | a18138132bd35df3ebbcc6d39091c6e713892b53 (patch) | |
| tree | 304d8b2d8c77873ce72b0567b4100be5ed83e160 | |
| parent | f989a1c8ec5d25586c937cc39948b032e07cf1c0 (diff) | |
| download | gdb-a18138132bd35df3ebbcc6d39091c6e713892b53.zip gdb-a18138132bd35df3ebbcc6d39091c6e713892b53.tar.gz gdb-a18138132bd35df3ebbcc6d39091c6e713892b53.tar.bz2 | |
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.
git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/trunk@207002 138bc75d-0d04-0410-961f-82ee72b054a4
| -rw-r--r-- | libdecnumber/ChangeLog | 5 | ||||
| -rw-r--r-- | libdecnumber/decNumberLocal.h | 7 |
2 files changed, 8 insertions, 4 deletions
diff --git a/libdecnumber/ChangeLog b/libdecnumber/ChangeLog index 52b5178..65d20fc 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. + 2013-10-15 David Malcolm <dmalcolm@redhat.com> * Makefile.in (PICFLAG): New. diff --git a/libdecnumber/decNumberLocal.h b/libdecnumber/decNumberLocal.h index c8dbafc..b0e404d 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] */ |