]> granicus.if.org Git - clang/commitdiff
reduce indentation with an early exit.
authorChris Lattner <sabre@nondot.org>
Sun, 29 Mar 2009 14:02:43 +0000 (14:02 +0000)
committerChris Lattner <sabre@nondot.org>
Sun, 29 Mar 2009 14:02:43 +0000 (14:02 +0000)
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@67997 91177308-0d34-0410-b5e6-96231b3b80d8

lib/Parse/ParseDeclCXX.cpp

index 4201e1cb1eb2aee62181f14c513d0ee324a37697..c50a033e561347b105037eee0f1b3a22a97d6dec 100644 (file)
@@ -64,36 +64,34 @@ Parser::DeclPtrTy Parser::ParseNamespace(unsigned Context) {
     // FIXME: Verify no attributes were present.
     return ParseNamespaceAlias(NamespaceLoc, IdentLoc, Ident);
   
-  if (Tok.is(tok::l_brace)) {
-    SourceLocation LBrace = ConsumeBrace();
+  if (Tok.isNot(tok::l_brace)) {
+    Diag(Tok, Ident ? diag::err_expected_lbrace : 
+         diag::err_expected_ident_lbrace);
+    return DeclPtrTy();
+  }
+  
+  SourceLocation LBrace = ConsumeBrace();
 
-    // Enter a scope for the namespace.
-    ParseScope NamespaceScope(this, Scope::DeclScope);
+  // Enter a scope for the namespace.
+  ParseScope NamespaceScope(this, Scope::DeclScope);
 
-    DeclPtrTy NamespcDecl =
-      Actions.ActOnStartNamespaceDef(CurScope, IdentLoc, Ident, LBrace);
+  DeclPtrTy NamespcDecl =
+    Actions.ActOnStartNamespaceDef(CurScope, IdentLoc, Ident, LBrace);
 
-    PrettyStackTraceActionsDecl CrashInfo(NamespcDecl, NamespaceLoc, Actions,
-                                          PP.getSourceManager(),
-                                          "parsing namespace");
-    
-    while (Tok.isNot(tok::r_brace) && Tok.isNot(tok::eof))
-      ParseExternalDeclaration();
-    
-    // Leave the namespace scope.
-    NamespaceScope.Exit();
+  PrettyStackTraceActionsDecl CrashInfo(NamespcDecl, NamespaceLoc, Actions,
+                                        PP.getSourceManager(),
+                                        "parsing namespace");
+  
+  while (Tok.isNot(tok::r_brace) && Tok.isNot(tok::eof))
+    ParseExternalDeclaration();
+  
+  // Leave the namespace scope.
+  NamespaceScope.Exit();
 
-    SourceLocation RBrace = MatchRHSPunctuation(tok::r_brace, LBrace);
-    Actions.ActOnFinishNamespaceDef(NamespcDecl, RBrace);
+  SourceLocation RBrace = MatchRHSPunctuation(tok::r_brace, LBrace);
+  Actions.ActOnFinishNamespaceDef(NamespcDecl, RBrace);
 
-    return NamespcDecl;
-    
-  } else {
-    Diag(Tok, Ident ? diag::err_expected_lbrace : 
-                      diag::err_expected_ident_lbrace);
-  }
-  
-  return DeclPtrTy();
+  return NamespcDecl;
 }
 
 /// ParseNamespaceAlias - Parse the part after the '=' in a namespace