re PR c++/9823 (ICE in sort_mem_initializers)

	PR c++/9823
	* cp-tree.h (begin_mem_initializers): Remove.
	* parser.c (cp_parser_mem_initializer_list): Inline it here.
	Do not call finish_mem_initializers if not in a constructor.
	(cp_parser_class_head): Fix typo in error message.
	* semantics.c (begin_mem_initializers): Remove.
	* testsuite/g++.dg/parser/constructor1.C: New test.

	PR c++/9823
	* g++.dg/parser/constructor1.C: New test.

From-SVN: r63999

gcc/cp/ChangeLog

 2003-03-08  Mark Mitchell  <mark@codesourcery.com>
 
+	PR c++/9823
+	* cp-tree.h (begin_mem_initializers): Remove.
+	* parser.c (cp_parser_mem_initializer_list): Inline it here.
+	Do not call finish_mem_initializers if not in a constructor.
+	(cp_parser_class_head): Fix typo in error message.
+	* semantics.c (begin_mem_initializers): Remove.
+	* testsuite/g++.dg/parser/constructor1.C: New test.
+
 	PR c++/9809
 	* call.c (add_function_candidate): Skip builtin fuctions that have
 	not yet been declared.

gcc/cp/cp-tree.h

@@ -4205,7 +4205,6 @@ extern void expand_body                         (tree);
 extern tree nullify_returns_r		      (tree *, int *, void *);
 extern void do_pushlevel                        (scope_kind);
 extern tree do_poplevel                         (void);
-extern void begin_mem_initializers              (void);
 extern void finish_mem_initializers             (tree);
 extern void setup_vtbl_ptr			(tree, tree);
 extern void clear_out_block                     (void);

gcc/cp/parser.c

@@ -7189,7 +7189,8 @@ cp_parser_mem_initializer_list (cp_parser* parser)
 
   /* Let the semantic analysis code know that we are starting the
      mem-initializer-list.  */
-  begin_mem_initializers ();
+  if (!DECL_CONSTRUCTOR_P (current_function_decl))
+    error ("only constructors take base initializers");
 
   /* Loop through the list.  */
   while (true)
@@ -7212,7 +7213,8 @@ cp_parser_mem_initializer_list (cp_parser* parser)
     }
 
   /* Perform semantic analysis.  */
-  finish_mem_initializers (mem_initializer_list);
+  if (DECL_CONSTRUCTOR_P (current_function_decl))
+    finish_mem_initializers (mem_initializer_list);
 }
 
 /* Parse a mem-initializer.
@@ -11789,7 +11791,7 @@ cp_parser_class_head (cp_parser* parser,
 	 class was originally declared, the program is invalid.  */
       if (scope && !is_ancestor (scope, CP_DECL_CONTEXT (type)))
 	{
-	  error ("declaration of `%D' in  `%D' which does not "
+	  error ("declaration of `%D' in `%D' which does not "
 		 "enclose `%D'", type, scope, nested_name_specifier);
 	  return NULL_TREE;
 	}

gcc/cp/semantics.c

@@ -1142,15 +1142,6 @@ finish_eh_cleanup (cleanup)
   add_stmt (r);
 }
 
-/* Begin processing a mem-initializer-list.  */
-
-void
-begin_mem_initializers ()
-{
-  if (! DECL_CONSTRUCTOR_P (current_function_decl))
-    error ("only constructors take base initializers");
-}
-
 /* The MEM_INITS is a list of mem-initializers, in reverse of the
    order they were written by the user.  Each node is as for
    emit_mem_initializers.  */

gcc/testsuite/ChangeLog

+2003-03-08  Mark Mitchell  <mark@codesourcery.com>
+
+	PR c++/9823
+	* g++.dg/parser/constructor1.C: New test.
+
 2003-03-08  Hans-Peter Nilsson  <hp@bitrange.com>
 
 	* gcc.c-torture/execute/20020720-1.x: Add xfail for cris-*-*.

gcc/testsuite/g++.dg/parse/constructor1.C

+ACE_Process_Descriptor::ACE_Process_Descriptor () : // { dg-error "" }
+  process_ (0)
+{
+}