diff mbox

[C++] kill namespace walking

Message ID cc090b82-0bfc-ba01-b02d-486cb231cfa7@acm.org
State New
Headers show

Commit Message

Nathan Sidwell May 5, 2017, 3:07 p.m. UTC 
And with static_decls no longer being namespace-specific, we can kill 
the namespace walker function. (the spelling suggester doesn't use it, 
and my to-be-committed reworking will allow that to be improved)

nathan
diff mbox

Patch

2017-05-05  Nathan Sidwell  <nathan@acm.org>

	Kill walk_namespaces.
	* cp-tree.h (walk_namespaces_fn, walk_namespaces): Delete.
	* decl.c (walk_namespaces_r, walk_namespaces): Delete.

Index: cp-tree.h
===================================================================
--- cp-tree.h	(revision 247636)
+++ cp-tree.h	(working copy)
@@ -5902,9 +5902,6 @@  extern void revert_static_member_fn		(tr
 extern void fixup_anonymous_aggr		(tree);
 extern tree compute_array_index_type		(tree, tree, tsubst_flags_t);
 extern tree check_default_argument		(tree, tree, tsubst_flags_t);
-typedef int (*walk_namespaces_fn)		(tree, void *);
-extern int walk_namespaces			(walk_namespaces_fn,
-						 void *);
 extern int wrapup_namespace_globals		();
 extern tree create_implicit_typedef		(tree, tree);
 extern int local_variable_p			(const_tree);
Index: decl.c
===================================================================
--- decl.c	(revision 247636)
+++ decl.c	(working copy)
@@ -78,7 +78,6 @@  static void check_for_uninitialized_cons
 static tree local_variable_p_walkfn (tree *, int *, void *);
 static const char *tag_name (enum tag_types);
 static tree lookup_and_check_tag (enum tag_types, tree, tag_scope, bool);
-static int walk_namespaces_r (tree, walk_namespaces_fn, void *);
 static void maybe_deduce_size_from_array_init (tree, tree);
 static void layout_var_decl (tree);
 static tree check_initializer (tree, tree, int, vec<tree, va_gc> **);
@@ -879,32 +878,6 @@  poplevel (int keep, int reverse, int fun
   return block;
 }
 
-/* Walk all the namespaces contained NAMESPACE, including NAMESPACE
-   itself, calling F for each.  The DATA is passed to F as well.  */
-
-static int
-walk_namespaces_r (tree name_space, walk_namespaces_fn f, void* data)
-{
-  int result = 0;
-  tree current = NAMESPACE_LEVEL (name_space)->namespaces;
-
-  result |= (*f) (name_space, data);
-
-  for (; current; current = DECL_CHAIN (current))
-    result |= walk_namespaces_r (current, f, data);
-
-  return result;
-}
-
-/* Walk all the namespaces, calling F for each.  The DATA is passed to
-   F as well.  */
-
-int
-walk_namespaces (walk_namespaces_fn f, void* data)
-{
-  return walk_namespaces_r (global_namespace, f, data);
-}
-
 /* Call wrapup_globals_declarations for the globals in NAMESPACE.  */
 /* Diagnose odr-used extern inline variables without definitions
    in the current TU.  */