c-opts.c (c_common_handle_option): -Wnontemplate-friend...

2007-10-09  Manuel Lopez-Ibanez  <manu@gcc.gnu.org>

	* c-opts.c (c_common_handle_option): -Wnontemplate-friend,
	-Wwrite-strings and -Wmultichar are enabled by default, so Wall
	enabling them is redundant. Don't check two times for
	c_dialect_cxx.

From-SVN: r129183

diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index 2c65ee61f206e24044a03bd93fd684fa32563eb1..1aa5aa39c7fe751e263c2026391535cf153ec15a 100644 (file)
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,10 @@
+2007-10-09  Manuel Lopez-Ibanez  <manu@gcc.gnu.org>
+
+       * c-opts.c (c_common_handle_option): -Wnontemplate-friend,
+       -Wwrite-strings and -Wmultichar are enabled by default, so Wall
+       enabling them is redundant. Don't check two times for
+       c_dialect_cxx.
+       
 2007-10-09  H.J. Lu  <hongjiu.lu@intel.com>
 
        * ra-conflict.c (record_one_conflict_between_regnos): Revert

diff --git a/gcc/c-opts.c b/gcc/c-opts.c
index 80c01a01c1d86470ae5658a7c50eee88e7f2ff59..16710b6a8259b678b7e28a13f6b71d816ae6427f 100644 (file)
--- a/gcc/c-opts.c
+++ b/gcc/c-opts.c
@@ -400,8 +400,6 @@ c_common_handle_option (size_t scode, const char *arg, int value)
       warn_parentheses = value;
       warn_return_type = value;
       warn_sequence_point = value;     /* Was C only.  */
-      if (c_dialect_cxx ())
-       warn_sign_compare = value;
       warn_switch = value;
       set_Wstrict_aliasing (value);
       warn_address = value;
@@ -425,17 +423,14 @@ c_common_handle_option (size_t scode, const char *arg, int value)
       else
        {
          /* C++-specific warnings.  */
+          warn_sign_compare = value;
          warn_reorder = value;
-         warn_nontemplate_friend = value;
           warn_cxx0x_compat = value;
-          if (value > 0)
-            warn_write_strings = true;
        }
 
       cpp_opts->warn_trigraphs = value;
       cpp_opts->warn_comments = value;
       cpp_opts->warn_num_sign_change = value;
-      cpp_opts->warn_multichar = value;        /* Was C++ only.  */
 
       if (warn_pointer_sign == -1)
        warn_pointer_sign = 1;