Remove search_parents parameter from d_lookup_symbol_imports.
authorIain Buclaw <ibuclaw@gdcproject.org>
Sun, 21 Feb 2016 20:38:10 +0000 (21:38 +0100)
committerIain Buclaw <ibuclaw@gdcproject.org>
Sun, 21 Feb 2016 20:38:34 +0000 (21:38 +0100)
gdb/ChangeLog:

* d-namespace.c (d_lookup_symbol_imports): Remove argument
'search_parents'.  All callers updated.

gdb/ChangeLog
gdb/d-namespace.c

index 274d98b0f0fe1d0422679b8964e8840eb73cfedb..3d442cc65792bd93546c8c4080e8a7c74d99a2ec 100644 (file)
@@ -1,3 +1,8 @@
+2016-02-21  Iain Buclaw  <ibuclaw@gdcproject.org>
+
+       * d-namespace.c (d_lookup_symbol_imports): Remove argument
+       'search_parents'.  All callers updated.
+
 2016-02-18  Marcin Koƛcielnicki  <koriakin@0x04.net>
 
        * s390-linux-tdep.c (s390_guess_tracepoint_registers): New function.
index bcd0f45e7521606fb937375e2f78965f54e7c1d2..2c8b36c988fbb20e9e634a9963a5141a723b22e9 100644 (file)
@@ -388,31 +388,15 @@ reset_directive_searched (void *data)
 }
 
 /* Search for NAME by applying all import statements belonging to
-   BLOCK which are applicable in SCOPE.
-
-   If SEARCH_PARENTS the search will include imports which are
-   applicable in parents of SCOPE.
-   Example:
-
-     module A;
-     import X;
-     void B() {
-       import Y;
-     }
-
-   If SCOPE is "A.B" and SEARCH_PARENTS is true, the imports of
-   modules X and Y will be considered.  If SEARCH_PARENTS is false
-   only the import of Y is considered.  */
+   BLOCK which are applicable in SCOPE.  */
 
 static struct block_symbol
 d_lookup_symbol_imports (const char *scope, const char *name,
                         const struct block *block,
-                        const domain_enum domain,
-                        const int search_parents)
+                        const domain_enum domain)
 {
   struct using_direct *current;
   struct block_symbol sym;
-  int directive_match;
   struct cleanup *searched_cleanup;
 
   /* First, try to find the symbol in the given module.  */
@@ -430,18 +414,9 @@ d_lookup_symbol_imports (const char *scope, const char *name,
        current = current->next)
     {
       const char **excludep;
-      int len = strlen (current->import_dest);
-
-      directive_match = (search_parents
-                        ? (strncmp (scope, current->import_dest, len) == 0
-                           && (len == 0
-                               || scope[len] == '.'
-                               || scope[len] == '\0'))
-                        : strcmp (scope, current->import_dest) == 0);
-
-      /* If the import destination is the current scope or one of its
-        ancestors then it is applicable.  */
-      if (directive_match && !current->searched)
+
+      /* If the import destination is the current scope then search it.  */
+      if (!current->searched && strcmp (scope, current->import_dest) == 0)
        {
          /* Mark this import as searched so that the recursive call
             does not search it again.  */
@@ -554,7 +529,7 @@ d_lookup_symbol_module (const char *scope, const char *name,
      blocks.  */
   while (block != NULL)
     {
-      sym = d_lookup_symbol_imports (scope, name, block, domain, 1);
+      sym = d_lookup_symbol_imports (scope, name, block, domain);
 
       if (sym.symbol != NULL)
        return sym;