c-common.c: Change warning messages to say `comparison is always true' or `comparison...

        * 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

diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index a6b9b8469836cf6121de1efaaaf772b86f1ddb8e..845216fbf3ae46beeb47f66a2bb7fa8aaa3507c4 100644 (file)
--- 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 f45b5662020f435c007777eb85024eba9177104f..653af682d70d9ef54f4d4dde2cfb235b1765ce76 100644 (file)
--- 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;