]> granicus.if.org Git - clang/commitdiff
Fixed a recent test regression from last night (r51976).
authorSteve Naroff <snaroff@apple.com>
Thu, 5 Jun 2008 15:03:27 +0000 (15:03 +0000)
committerSteve Naroff <snaroff@apple.com>
Thu, 5 Jun 2008 15:03:27 +0000 (15:03 +0000)
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@52006 91177308-0d34-0410-b5e6-96231b3b80d8

lib/Sema/SemaDeclObjC.cpp

index 3117f87f7bad4b2ce79d1a6f6189bc0d9a70bf63..84a8e82750961028d6717e649e3719adfac1b540 100644 (file)
@@ -408,19 +408,19 @@ Sema::DeclTy *Sema::ActOnStartCategoryInterface(
   /// Check that class of this category is already completely declared.
   if (!IDecl || IDecl->isForwardDecl())
     Diag(ClassLoc, diag::err_undef_interface, ClassName->getName());
-  else if (CategoryName) {
+  else {
     /// Check for duplicate interface declaration for this category
     ObjCCategoryDecl *CDeclChain;
     for (CDeclChain = IDecl->getCategoryList(); CDeclChain;
          CDeclChain = CDeclChain->getNextClassCategory()) {
-      if (CDeclChain->getIdentifier() == CategoryName) {
+      if (CategoryName && CDeclChain->getIdentifier() == CategoryName) {
         Diag(CategoryLoc, diag::warn_dup_category_def, ClassName->getName(),
              CategoryName->getName());
         break;
       }
     }
-  if (!CDeclChain)
-    CDecl->insertNextClassCategory();
+    if (!CDeclChain)
+      CDecl->insertNextClassCategory();
   }
 
   if (NumProtoRefs) {