diff mbox

[5/5] cp-namespace.c cleanup pass: remove redundancies in cp_lookup_symbol_nonlocal

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

Commit Message

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

This patch removes some redundancies in cp_lookup_symbol_nonlocal.

It also makes cp_lookup_symbol_nonlocal way easier for me to understand.
cp_lookup_symbol_nonlocal basically just calls two functions:

  sym = lookup_namespace_scope (name, block,
				domain, scope, 0);
  if (sym != NULL)
    return sym;

  return cp_lookup_symbol_namespace (scope, name,
				     block, domain);

One has to dig to understand what's going on here because
the names lookup_namespace_scope and cp_lookup_symbol_namespace
are quite similar and a bit opaque.

The redundancy is that the first thing cp_lookup_symbol_namespace does
is repeat what we've already done in the call to lookup_namespace_scope.

So this patch just factors out the cp_lookup_symbol_via_imports loop
and calls that from both cp_lookup_symbol_namespace and
cp_lookup_symbol_nonlocal.

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

	* cp-namespace.c (cp_lookup_symbol_via_all_imports): New function.
	(cp_lookup_symbol_namespace): Call it.
	(cp_lookup_symbol_nonlocal): Ditto.
diff mbox

Patch

diff --git a/gdb/cp-namespace.c b/gdb/cp-namespace.c
index 045abc5..1caa3a1 100644
--- a/gdb/cp-namespace.c
+++ b/gdb/cp-namespace.c
@@ -646,12 +646,35 @@  cp_lookup_symbol_imports_or_template (const char *scope,
   return cp_lookup_symbol_via_imports (scope, name, block, domain, 0, 1, 1);
 }
 
+/* Search for NAME by applying relevant import statements belonging to BLOCK
+   and its parents.  SCOPE is the namespace scope of the context in which the
+   search is being evaluated.  */
+
+static struct symbol *
+cp_lookup_symbol_via_all_imports (const char *scope, const char *name,
+				  const struct block *block,
+				  const domain_enum domain)
+{
+  struct symbol *sym;
+
+  while (block != NULL)
+    {
+      sym = cp_lookup_symbol_via_imports (scope, name, block, domain, 0, 0, 1);
+      if (sym)
+	return sym;
+
+      block = BLOCK_SUPERBLOCK (block);
+    }
+
+  return NULL;
+}
+
 /* Searches for NAME in the current namespace, and by applying
    relevant import statements belonging to BLOCK and its parents.
    SCOPE is the namespace scope of the context in which the search is
    being evaluated.  */
 
-struct symbol*
+struct symbol *
 cp_lookup_symbol_namespace (const char *scope,
                             const char *name,
                             const struct block *block,
@@ -660,25 +683,12 @@  cp_lookup_symbol_namespace (const char *scope,
   struct symbol *sym;
 
   /* First, try to find the symbol in the given namespace.  */
-  sym = cp_lookup_symbol_in_namespace (scope, name,
-				       block, domain, 1);
+  sym = cp_lookup_symbol_in_namespace (scope, name, block, domain, 1);
   if (sym != NULL)
     return sym;
 
-  /* Search for name in namespaces imported to this and parent
-     blocks.  */
-  while (block != NULL)
-    {
-      sym = cp_lookup_symbol_via_imports (scope, name, block,
-					  domain, 0, 0, 1);
-
-      if (sym)
-	return sym;
-
-      block = BLOCK_SUPERBLOCK (block);
-    }
-
-  return NULL;
+  /* Search for name in namespaces imported to this and parent blocks.  */
+  return cp_lookup_symbol_via_all_imports (scope, name, block, domain);
 }
 
 /* Lookup NAME at namespace scope (or, in C terms, in static and
@@ -749,13 +759,14 @@  cp_lookup_symbol_nonlocal (const char *name,
   struct symbol *sym;
   const char *scope = block_scope (block);
 
-  sym = lookup_namespace_scope (name, block,
-				domain, scope, 0);
+  /* First, try to find the symbol in the given namespace, and all
+     containing namespaces.  */
+  sym = lookup_namespace_scope (name, block, domain, scope, 0);
   if (sym != NULL)
     return sym;
 
-  return cp_lookup_symbol_namespace (scope, name,
-				     block, domain);
+  /* Search for name in namespaces imported to this and parent blocks.  */
+  return cp_lookup_symbol_via_all_imports (scope, name, block, domain);
 }
 
 /* Search through the base classes of PARENT_TYPE for a base class