* class.c (add_method, check_field_decl): Fix format specifier.

	* decl.c (duplicate_decls, pushdecl, check_goto,
	fixup_anonymous_aggr, maybe_commonize_var, grokdeclarator,
	start_enum): Likewise.
	* decl2.c (ambiguous_decl): Likewise.
	* pt.c (redeclare_class_template): Likewise.

git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/trunk@68878 138bc75d-0d04-0410-961f-82ee72b054a4

diff --git a/gcc/cp/ChangeLog b/gcc/cp/ChangeLog
index 9708304..c0b47dc 100644 (file)
--- a/gcc/cp/ChangeLog
+++ b/gcc/cp/ChangeLog
@@ -1,3 +1,12 @@
+2003-07-03  Kaveh R. Ghazi  <ghazi@caip.rutgers.edu>
+
+       * class.c (add_method, check_field_decl): Fix format specifier.
+       * decl.c (duplicate_decls, pushdecl, check_goto,
+       fixup_anonymous_aggr, maybe_commonize_var, grokdeclarator,
+       start_enum): Likewise.
+       * decl2.c (ambiguous_decl): Likewise.
+       * pt.c (redeclare_class_template): Likewise.
+
 2003-07-02  Nathan Sidwell  <nathan@codesourcery.com>
 
        PR c++/10219

diff --git a/gcc/cp/class.c b/gcc/cp/class.c
index b5eb245..09688b6 100644 (file)
--- a/gcc/cp/class.c
+++ b/gcc/cp/class.c
@@ -927,7 +927,7 @@ add_method (tree type, tree method, int error_p)
              else
                {
                  cp_error_at ("`%#D' and `%#D' cannot be overloaded",
-                              method, fn, method);
+                              method, fn);
 
                  /* We don't call duplicate_decls here to merge
                     the declarations because that will confuse
@@ -2961,7 +2961,7 @@ check_field_decl (tree field,
       /* `build_class_init_list' does not recognize
         non-FIELD_DECLs.  */
       if (TREE_CODE (t) == UNION_TYPE && any_default_members != 0)
-       cp_error_at ("multiple fields in union `%T' initialized");
+       error ("multiple fields in union `%T' initialized", t);
       *any_default_members = 1;
     }
 }

diff --git a/gcc/cp/decl.c b/gcc/cp/decl.c
index 863a21d..732ada2 100644 (file)
--- a/gcc/cp/decl.c
+++ b/gcc/cp/decl.c
@@ -3138,7 +3138,8 @@ duplicate_decls (tree newdecl, tree olddecl)
        {
          /* Prototype decl follows defn w/o prototype.  */
          cp_warning_at ("prototype for `%#D'", newdecl);
-         cp_warning_at ("follows non-prototype definition here", olddecl);
+         warning ("%Hfollows non-prototype definition here",
+                  &DECL_SOURCE_LOCATION (olddecl));
        }
       else if (TREE_CODE (olddecl) == FUNCTION_DECL
               && DECL_LANGUAGE (newdecl) != DECL_LANGUAGE (olddecl))
@@ -3195,8 +3196,8 @@ duplicate_decls (tree newdecl, tree olddecl)
            {
              warning ("`%#D' was used before it was declared inline",
                          newdecl);
-             cp_warning_at ("previous non-inline declaration here",
-                            olddecl);
+             warning ("%Hprevious non-inline declaration here",
+                      &DECL_SOURCE_LOCATION (olddecl));
            }
        }
     }
@@ -3888,7 +3889,7 @@ pushdecl (tree x)
              && TREE_CODE (decl) == TREE_CODE (x)
              && !same_type_p (TREE_TYPE (x), TREE_TYPE (decl)))
            {
-             pedwarn ("type mismatch with previous external decl", x);
+             pedwarn ("type mismatch with previous external decl of `%#D'", x);
              cp_pedwarn_at ("previous external decl of `%#D'", decl);
            }
        }
@@ -4925,7 +4926,7 @@ check_goto (tree decl)
 
       if (u > 1 && DECL_ARTIFICIAL (b))
        /* Can't skip init of __exception_info.  */
-       cp_error_at ("  enters catch block", b);
+       error ("%H  enters catch block", &DECL_SOURCE_LOCATION (b));
       else if (u > 1)
        cp_error_at ("  skips initialization of `%#D'", b);
       else
@@ -6692,7 +6693,8 @@ fixup_anonymous_aggr (tree t)
 
   /* ISO C++ 9.5.3.  Anonymous unions may not have function members.  */
   if (TYPE_METHODS (t))
-    cp_error_at ("an anonymous union cannot have function members", t);
+    error ("%Han anonymous union cannot have function members",
+          &DECL_SOURCE_LOCATION (TYPE_MAIN_DECL (t)));
 
   /* Anonymous aggregates cannot have fields with ctors, dtors or complex
      assignment operators (because they cannot have these methods themselves).
@@ -7393,7 +7395,8 @@ maybe_commonize_var (tree decl)
              TREE_PUBLIC (decl) = 0;
              DECL_COMMON (decl) = 0;
              cp_warning_at ("sorry: semantics of inline function static data `%#D' are wrong (you'll wind up with multiple copies)", decl);
-             cp_warning_at ("  you can work around this by removing the initializer", decl);
+             warning ("%H  you can work around this by removing the initializer",
+                      &DECL_SOURCE_LOCATION (decl));
            }
        }
     }
@@ -11140,8 +11143,8 @@ grokdeclarator (tree declarator,
        {
          decl = build_decl (TYPE_DECL, declarator, type);
          if (in_namespace || ctype)
-           cp_error_at ("typedef name may not be a nested-name-specifier",
-                        decl);
+           error ("%Htypedef name may not be a nested-name-specifier",
+                  &DECL_SOURCE_LOCATION (decl));
          if (!current_function_decl)
            DECL_CONTEXT (decl) = FROB_CONTEXT (current_namespace);
        }
@@ -11187,7 +11190,8 @@ grokdeclarator (tree declarator,
          if (ctype == NULL_TREE)
            {
              if (TREE_CODE (type) != METHOD_TYPE)
-               cp_error_at ("invalid type qualifier for non-member function type", decl);
+               error ("%Hinvalid type qualifier for non-member function type",
+                      &DECL_SOURCE_LOCATION (decl));
              else
                ctype = TYPE_METHOD_BASETYPE (type);
            }
@@ -13014,7 +13018,8 @@ start_enum (tree name)
   if (enumtype != NULL_TREE && TREE_CODE (enumtype) == ENUMERAL_TYPE)
     {
       error ("multiple definition of `%#T'", enumtype);
-      cp_error_at ("previous definition here", enumtype);
+      error ("%Hprevious definition here",
+            &DECL_SOURCE_LOCATION (TYPE_MAIN_DECL (enumtype)));
       /* Clear out TYPE_VALUES, and start again.  */
       TYPE_VALUES (enumtype) = NULL_TREE;
     }

diff --git a/gcc/cp/decl2.c b/gcc/cp/decl2.c
index c3b7cd0..faa36f5 100644 (file)
--- a/gcc/cp/decl2.c
+++ b/gcc/cp/decl2.c
@@ -3664,8 +3664,10 @@ ambiguous_decl (tree name, cxx_binding *old, cxx_binding *new, int flags)
       if (flags & LOOKUP_COMPLAIN)
         {
           error ("`%D' denotes an ambiguous type",name);
-          cp_error_at ("  first type here", BINDING_TYPE (old));
-          cp_error_at ("  other type here", type);
+          error ("%H  first type here",
+                &DECL_SOURCE_LOCATION (TYPE_MAIN_DECL (BINDING_TYPE (old))));
+          error ("%H  other type here",
+                &DECL_SOURCE_LOCATION (TYPE_MAIN_DECL (type)));
         }
     }
   return old;

diff --git a/gcc/cp/pt.c b/gcc/cp/pt.c
index 607bdd1..11f0bcd 100644 (file)
--- a/gcc/cp/pt.c
+++ b/gcc/cp/pt.c
@@ -2984,7 +2984,8 @@ redeclare_class_template (tree type, tree parms)
             A template-parameter may not be given default arguments
             by two different declarations in the same scope.  */
          error ("redefinition of default argument for `%#D'", parm);
-         cp_error_at ("  original definition appeared here", tmpl_parm);
+         error ("%H  original definition appeared here",
+                &DECL_SOURCE_LOCATION (tmpl_parm));
          return;
        }