diff options
| author | Ian Lance Taylor <iant@google.com> | 2007-01-02 17:49:07 +0000 |
|---|---|---|
| committer | Ian Lance Taylor <ian@gcc.gnu.org> | 2007-01-02 17:49:07 +0000 |
| commit | fbc8d2d30e4c1987e85e2d3fda043f2ae24c7ad1 (patch) | |
| tree | 8c417c46fb300ca0e55218e9d49e86f6e77ea715 /gcc/c-common.c | |
| parent | b1ed4cb43c9b43a57b28d35c46062607af6e6a64 (diff) | |
| download | gcc-fbc8d2d30e4c1987e85e2d3fda043f2ae24c7ad1.zip gcc-fbc8d2d30e4c1987e85e2d3fda043f2ae24c7ad1.tar.gz gcc-fbc8d2d30e4c1987e85e2d3fda043f2ae24c7ad1.tar.bz2 | |
c-common.c (c_common_truthvalue_conversion): When warning about using an assignment as a truth value, set TREE_NO_WARNING.
./: * c-common.c (c_common_truthvalue_conversion): When warning about
using an assignment as a truth value, set TREE_NO_WARNING.
cp/:
* semantics.c (maybe_convert_cond): Optionally warn when using an
assignment as a condition.
* typeck.c (convert_for_assignment): Optionally warn about
assigning the result of an assignment to a bool.
testsuite/:
* g++.dg/warn/Wparentheses-22.C: New test.
* g++.dg/warn/Wparentheses-23.C: New test.
From-SVN: r120348
Diffstat (limited to 'gcc/c-common.c')
| -rw-r--r-- | gcc/c-common.c | 12 |
1 files changed, 8 insertions, 4 deletions
diff --git a/gcc/c-common.c b/gcc/c-common.c index 0658141..cca1490 100644 --- a/gcc/c-common.c +++ b/gcc/c-common.c @@ -1,6 +1,6 @@ /* Subroutines shared by all languages that are variants of C. Copyright (C) 1992, 1993, 1994, 1995, 1996, 1997, 1998, 1999, 2000, - 2001, 2002, 2003, 2004, 2005 Free Software Foundation, Inc. + 2001, 2002, 2003, 2004, 2005, 2006, 2007 Free Software Foundation, Inc. This file is part of GCC. @@ -2726,9 +2726,13 @@ c_common_truthvalue_conversion (tree expr) break; case MODIFY_EXPR: - if (!TREE_NO_WARNING (expr)) - warning (OPT_Wparentheses, - "suggest parentheses around assignment used as truth value"); + if (!TREE_NO_WARNING (expr) + && warn_parentheses) + { + warning (OPT_Wparentheses, + "suggest parentheses around assignment used as truth value"); + TREE_NO_WARNING (expr) = 1; + } break; default: |