diff mbox

[4/5] cp-namespace.c cleanup pass: remove redundant calls to cp_lookup_symbol_in_namespace

Message ID m3vbldh1yn.fsf@seba.sebabeach.org
State New
Headers show

Commit Message

Doug Evans Dec. 15, 2014, 6:16 a.m. UTC
Hi.

This patch removes some redundant calls to cp_lookup_symbol_in_namespace.

Note: This assumes this patch has been applied:
https://sourceware.org/ml/gdb-patches/2014-12/msg00331.html

The first thing cp_lookup_symbol_via_imports does is this:

  /* First, try to find the symbol in the given namespace.  */
  if (!declaration_only)
     sym = cp_lookup_symbol_in_namespace (scope, name,
 					 block, domain, 1);

which is fine in the recursive case, but when called from
cp_lookup_symbol_namespace that's already been done.

2014-12-14  Doug Evans  <xdje42@gmail.com>

	* cp-namespace.c (cp_lookup_symbol_via_imports): New arg
	"search_scope_first".  All callers updated.
diff mbox

Patch

diff --git a/gdb/cp-namespace.c b/gdb/cp-namespace.c
index 6f9ab34..045abc5 100644
--- a/gdb/cp-namespace.c
+++ b/gdb/cp-namespace.c
@@ -429,13 +429,17 @@  reset_directive_searched (void *data)
 
    If SCOPE is "A::B" and SEARCH_PARENTS is true the imports of
    namespaces X and Y will be considered.  If SEARCH_PARENTS is false
-   only the import of Y is considered.  */
+   only the import of Y is considered.
+
+   SEARCH_SCOPE_FIRST is an internal implementation detail: Callers must
+   pass 0 for it.  Internally we pass 1 when recursing.  */
 
 static struct symbol *
 cp_lookup_symbol_via_imports (const char *scope,
 			      const char *name,
 			      const struct block *block,
 			      const domain_enum domain,
+			      const int search_scope_first,
 			      const int declaration_only,
 			      const int search_parents)
 {
@@ -445,8 +449,8 @@  cp_lookup_symbol_via_imports (const char *scope,
   int directive_match;
   struct cleanup *searched_cleanup;
 
-  /* First, try to find the symbol in the given namespace.  */
-  if (!declaration_only)
+  /* First, try to find the symbol in the given namespace if requested.  */
+  if (search_scope_first)
     sym = cp_lookup_symbol_in_namespace (scope, name,
 					 block, domain, 1);
 
@@ -535,7 +539,7 @@  cp_lookup_symbol_via_imports (const char *scope,
 		 towards the imported namespace.  */
 	      sym = cp_lookup_symbol_via_imports (current->import_src,
 						  name, block,
-						  domain, 0, 0);
+						  domain, 1, 0, 0);
 	    }
 	  current->searched = 0;
 	  discard_cleanups (searched_cleanup);
@@ -639,7 +643,7 @@  cp_lookup_symbol_imports_or_template (const char *scope,
 	}
     }
 
-  return cp_lookup_symbol_via_imports (scope, name, block, domain, 1, 1);
+  return cp_lookup_symbol_via_imports (scope, name, block, domain, 0, 1, 1);
 }
 
 /* Searches for NAME in the current namespace, and by applying
@@ -666,7 +670,7 @@  cp_lookup_symbol_namespace (const char *scope,
   while (block != NULL)
     {
       sym = cp_lookup_symbol_via_imports (scope, name, block,
-					  domain, 0, 1);
+					  domain, 0, 0, 1);
 
       if (sym)
 	return sym;