This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

FYI: reindent cp_lookup_symbol_imports


I am checking this in.

While reading cp_lookup_symbol_imports, I got a little confused because
the indentation was incorrect.  This patch fixes it.

Tom

2011-11-04  Tom Tromey  <tromey@redhat.com>

	* cp-namespace.c (cp_lookup_symbol_imports): Reindent.

Index: cp-namespace.c
===================================================================
RCS file: /cvs/src/src/gdb/cp-namespace.c,v
retrieving revision 1.55
diff -u -r1.55 cp-namespace.c
--- cp-namespace.c	20 Oct 2011 20:06:13 -0000	1.55
+++ cp-namespace.c	4 Nov 2011 16:36:03 -0000
@@ -370,72 +370,72 @@
          ancestors then it is applicable.  */
       if (directive_match && !current->searched)
 	{
-	/* Mark this import as searched so that the recursive call
-           does not search it again.  */
-	current->searched = 1;
-	searched_cleanup = make_cleanup (reset_directive_searched,
-					 current);
-
-	/* If there is an import of a single declaration, compare the
-	   imported declaration (after optional renaming by its alias)
-	   with the sought out name.  If there is a match pass
-	   current->import_src as NAMESPACE to direct the search
-	   towards the imported namespace.  */
-	if (current->declaration
-	    && strcmp (name, current->alias
-		       ? current->alias : current->declaration) == 0)
-	  sym = cp_lookup_symbol_in_namespace (current->import_src,
-					       current->declaration,
-					       block, domain);
-
-	/* If this is a DECLARATION_ONLY search or a symbol was found
-	   or this import statement was an import declaration, the
-	   search of this import is complete.  */
-        if (declaration_only || sym != NULL || current->declaration)
-          {
-            current->searched = 0;
-            discard_cleanups (searched_cleanup);
-
-            if (sym != NULL)
-              return sym;
-
-            continue;
-          }
-
-	/* Do not follow CURRENT if NAME matches its EXCLUDES.  */
-	for (excludep = current->excludes; *excludep; excludep++)
-	  if (strcmp (name, *excludep) == 0)
-	    break;
-	if (*excludep)
-	  {
-	    discard_cleanups (searched_cleanup);
-	    continue;
-	  }
-
-	if (current->alias != NULL
-	    && strcmp (name, current->alias) == 0)
-	  /* If the import is creating an alias and the alias matches
-	     the sought name.  Pass current->import_src as the NAME to
-	     direct the search towards the aliased namespace.  */
-	  {
-	    sym = cp_lookup_symbol_in_namespace (scope,
-						 current->import_src,
+	  /* Mark this import as searched so that the recursive call
+	     does not search it again.  */
+	  current->searched = 1;
+	  searched_cleanup = make_cleanup (reset_directive_searched,
+					   current);
+
+	  /* If there is an import of a single declaration, compare the
+	     imported declaration (after optional renaming by its alias)
+	     with the sought out name.  If there is a match pass
+	     current->import_src as NAMESPACE to direct the search
+	     towards the imported namespace.  */
+	  if (current->declaration
+	      && strcmp (name, current->alias
+			 ? current->alias : current->declaration) == 0)
+	    sym = cp_lookup_symbol_in_namespace (current->import_src,
+						 current->declaration,
 						 block, domain);
-	  }
-	else if (current->alias == NULL)
-	  {
-	    /* If this import statement creates no alias, pass
-               current->inner as NAMESPACE to direct the search
-               towards the imported namespace.  */
-	    sym = cp_lookup_symbol_imports (current->import_src,
-					    name, block,
-					    domain, 0, 0);
-	  }
-	current->searched = 0;
-	discard_cleanups (searched_cleanup);
 
-	if (sym != NULL)
-	  return sym;
+	  /* If this is a DECLARATION_ONLY search or a symbol was found
+	     or this import statement was an import declaration, the
+	     search of this import is complete.  */
+	  if (declaration_only || sym != NULL || current->declaration)
+	    {
+	      current->searched = 0;
+	      discard_cleanups (searched_cleanup);
+
+	      if (sym != NULL)
+		return sym;
+
+	      continue;
+	    }
+
+	  /* Do not follow CURRENT if NAME matches its EXCLUDES.  */
+	  for (excludep = current->excludes; *excludep; excludep++)
+	    if (strcmp (name, *excludep) == 0)
+	      break;
+	  if (*excludep)
+	    {
+	      discard_cleanups (searched_cleanup);
+	      continue;
+	    }
+
+	  if (current->alias != NULL
+	      && strcmp (name, current->alias) == 0)
+	    /* If the import is creating an alias and the alias matches
+	       the sought name.  Pass current->import_src as the NAME to
+	       direct the search towards the aliased namespace.  */
+	    {
+	      sym = cp_lookup_symbol_in_namespace (scope,
+						   current->import_src,
+						   block, domain);
+	    }
+	  else if (current->alias == NULL)
+	    {
+	      /* If this import statement creates no alias, pass
+		 current->inner as NAMESPACE to direct the search
+		 towards the imported namespace.  */
+	      sym = cp_lookup_symbol_imports (current->import_src,
+					      name, block,
+					      domain, 0, 0);
+	    }
+	  current->searched = 0;
+	  discard_cleanups (searched_cleanup);
+
+	  if (sym != NULL)
+	    return sym;
 	}
     }
 


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]