commit 2840008ec3711501aab8b5008f2df9b1a7a44dc7
author Chris Lattner <sabre@nondot.org> Mon Feb 16 19:25:52 2009 +0000
committer Chris Lattner <sabre@nondot.org> Mon Feb 16 19:25:52 2009 +0000
tree 097162421889bc405ee3df6ed065feb7545fad38
parent 715e8481aaefa26594ce6b1f613e7e9c8a66c5c6

code cleanup


git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@64651 91177308-0d34-0410-b5e6-96231b3b80d8

lib/Sema/SemaDeclObjC.cpp

diff --git a/lib/Sema/SemaDeclObjC.cpp b/lib/Sema/SemaDeclObjC.cpp
index 191ac4b..dc8ce06 100644
--- a/lib/Sema/SemaDeclObjC.cpp
+++ b/lib/Sema/SemaDeclObjC.cpp
@@ -15,7 +15,6 @@
 #include "clang/AST/ASTContext.h"
 #include "clang/AST/DeclObjC.h"
 #include "clang/Parse/DeclSpec.h"
-
 using namespace clang;
 
 /// ObjCActOnStartOfMethodDef - This routine sets up parameters; invisible
@@ -49,8 +48,7 @@
   // Introduce all of the other parameters into this scope.
   for (unsigned i = 0, e = MDecl->getNumParams(); i != e; ++i) {
     ParmVarDecl *PDecl = MDecl->getParamDecl(i);
-    IdentifierInfo *II = PDecl->getIdentifier();
-    if (II)
+    if (IdentifierInfo *II = PDecl->getIdentifier())
       PushOnScopeChains(PDecl, FnBodyScope);
   }
 }
@@ -814,20 +812,22 @@
   
   bool IncompleteImpl = false;
   for (ObjCInterfaceDecl::instmeth_iterator I = IDecl->instmeth_begin(),
-       E = IDecl->instmeth_end(); I != E; ++I)
-    if (!(*I)->isSynthesized() && !InsMap.count((*I)->getSelector()))
+       E = IDecl->instmeth_end(); I != E; ++I) {
+    if (!(*I)->isSynthesized() && !InsMap.count((*I)->getSelector())) {
       WarnUndefinedMethod(IMPDecl->getLocation(), *I, IncompleteImpl);
-    else {
-      ObjCMethodDecl *ImpMethodDecl = 
-        IMPDecl->getInstanceMethod((*I)->getSelector());
-      ObjCMethodDecl *IntfMethodDecl = 
-        IDecl->getInstanceMethod((*I)->getSelector());
-      assert(IntfMethodDecl && 
-             "IntfMethodDecl is null in ImplMethodsVsClassMethods");
-      // ImpMethodDecl may be null as in a @dynamic property.
-      if (ImpMethodDecl)
-        WarnConflictingTypedMethods(ImpMethodDecl, IntfMethodDecl);
+      continue;
     }
+    
+    ObjCMethodDecl *ImpMethodDecl = 
+      IMPDecl->getInstanceMethod((*I)->getSelector());
+    ObjCMethodDecl *IntfMethodDecl = 
+      IDecl->getInstanceMethod((*I)->getSelector());
+    assert(IntfMethodDecl && 
+           "IntfMethodDecl is null in ImplMethodsVsClassMethods");
+    // ImpMethodDecl may be null as in a @dynamic property.
+    if (ImpMethodDecl)
+      WarnConflictingTypedMethods(ImpMethodDecl, IntfMethodDecl);
+  }
       
   llvm::DenseSet<Selector> ClsMap;
   // Check and see if class methods in class interface have been
@@ -917,8 +917,8 @@
 /// ActOnForwardClassDeclaration - 
 Action::DeclTy *
 Sema::ActOnForwardClassDeclaration(SourceLocation AtClassLoc,
-                                   IdentifierInfo **IdentList, unsigned NumElts) 
-{
+                                   IdentifierInfo **IdentList,
+                                   unsigned NumElts) {
   llvm::SmallVector<ObjCInterfaceDecl*, 32> Interfaces;
   
   for (unsigned i = 0; i != NumElts; ++i) {