error.c (dump_scope): Don't add TFF_DECL_SPECIFIERS.

        * error.c (dump_scope): Don't add TFF_DECL_SPECIFIERS.
        (dump_function_decl): Always dump parms.

From-SVN: r49396

gcc/cp/ChangeLog

 2002-02-01  Jason Merrill  <jason@redhat.com>
 
-	PR c++/4872
-	* decl.c (finish_function): Warn about a non-void function with 
-	no return statement.
-
 	* error.c (dump_scope): Don't add TFF_DECL_SPECIFIERS.
 	(dump_function_decl): Always dump parms.
 

gcc/cp/error.c

@@ -140,8 +140,7 @@ dump_scope (scope, flags)
      tree scope;
      int flags;
 {
-  int f = ~TFF_RETURN_TYPE & (TFF_DECL_SPECIFIERS
-                              | (flags & (TFF_SCOPE | TFF_CHASE_TYPEDEF)));
+  int f = ~TFF_RETURN_TYPE & (flags & (TFF_SCOPE | TFF_CHASE_TYPEDEF));
 
   if (scope == NULL_TREE)
     return;
@@ -1137,7 +1136,7 @@ dump_function_decl (t, flags)
 
   dump_function_name (t, flags);
 
-  if (flags & TFF_DECL_SPECIFIERS) 
+  if (1)
     {
       dump_parameters (parmtypes, flags);
 

gcc/testsuite/g++.dg/template/local1.C

@@ -2,7 +2,7 @@
 // g(), because xref wanted the mangled name, which breaks inside a template.
 
 // Of course, the offending code is actually ill-formed anyway, so check
-// for the error.
+// for the error.  Also check that it's formatted properly.
 
 struct A
 {
@@ -14,7 +14,7 @@ template<class T> void A::f()
   struct B
   {
     void g() {}
-    static int x;		// { dg-error "static" "" }
+    static int x;	// { dg-error "static.*`int A::f\\(\\)::B::x'" "" }
   };
 }