diff options
| author | Zack Weinberg <zack@rabi.phys.columbia.edu> | 1998-11-19 19:40:56 +0000 |
|---|---|---|
| committer | Jeff Law <law@gcc.gnu.org> | 1998-11-19 12:40:56 -0700 |
| commit | 07be2a23729599fd76630d15e26e99d8e29c897e (patch) | |
| tree | fa266794ce38967b19c4f9ec6bcdd9dc4f2fdb26 | |
| parent | 74285159e4f348ab625717586a9dfa46ee0c8816 (diff) | |
| download | gcc-07be2a23729599fd76630d15e26e99d8e29c897e.zip gcc-07be2a23729599fd76630d15e26e99d8e29c897e.tar.gz gcc-07be2a23729599fd76630d15e26e99d8e29c897e.tar.bz2 | |
c-common.c: Change warning messages to say `comparison is always true' or `comparison is...
* c-common.c: Change warning messages to say `comparison is
always true' or `comparison is always false' instead of the
confusing `is always 0', `is always 1'.
From-SVN: r23723
| -rw-r--r-- | gcc/ChangeLog | 6 | ||||
| -rw-r--r-- | gcc/c-common.c | 12 |
2 files changed, 12 insertions, 6 deletions
diff --git a/gcc/ChangeLog b/gcc/ChangeLog index a6b9b84..845216f 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,9 @@ +1998-11-19 Zack Weinberg <zack@rabi.phys.columbia.edu> + + * c-common.c: Change warning messages to say `comparison is + always true' or `comparison is always false' instead of the + confusing `is always 0', `is always 1'. + Thu Nov 19 19:05:49 1998 Per Bothner <bothner@cygnus.com> * print-tree.c (print_node): After printing BLOCK or BIND_EXPR, diff --git a/gcc/c-common.c b/gcc/c-common.c index f45b566..653af68 100644 --- a/gcc/c-common.c +++ b/gcc/c-common.c @@ -2512,18 +2512,18 @@ shorten_compare (op0_ptr, op1_ptr, restype_ptr, rescode_ptr) /* This is the case of (char)x >?< 0x80, which people used to use expecting old C compilers to change the 0x80 into -0x80. */ if (val == boolean_false_node) - warning ("comparison is always 0 due to limited range of data type"); + warning ("comparison is always false due to limited range of data type"); if (val == boolean_true_node) - warning ("comparison is always 1 due to limited range of data type"); + warning ("comparison is always true due to limited range of data type"); } if (!min_lt && unsignedp0 && TREE_CODE (primop0) != INTEGER_CST) { /* This is the case of (unsigned char)x >?< -1 or < 0. */ if (val == boolean_false_node) - warning ("comparison is always 0 due to limited range of data type"); + warning ("comparison is always false due to limited range of data type"); if (val == boolean_true_node) - warning ("comparison is always 1 due to limited range of data type"); + warning ("comparison is always true due to limited range of data type"); } if (val != 0) @@ -2589,7 +2589,7 @@ shorten_compare (op0_ptr, op1_ptr, restype_ptr, rescode_ptr) && ! (TREE_CODE (primop0) == INTEGER_CST && ! TREE_OVERFLOW (convert (signed_type (type), primop0)))) - warning ("unsigned value >= 0 is always 1"); + warning ("comparison of unsigned expression >= 0 is always true"); value = boolean_true_node; break; @@ -2598,7 +2598,7 @@ shorten_compare (op0_ptr, op1_ptr, restype_ptr, rescode_ptr) && ! (TREE_CODE (primop0) == INTEGER_CST && ! TREE_OVERFLOW (convert (signed_type (type), primop0)))) - warning ("unsigned value < 0 is always 0"); + warning ("comparison of unsigned expression < 0 is always false"); value = boolean_false_node; break; |