[cfe-commits] r57798 - /cfe/trunk/lib/Parse/ParseObjc.cpp

[Chris Lattner]

Author: lattner
Date: Mon Oct 20 00:57:40 2008
New Revision: 57798

URL: http://llvm.org/viewvc/llvm-project?rev=57798&view=rev
Log:
fix some minor error recovery bugs in ParseObjCInterfaceDeclList
where it would reject @required in non-protocols, but then go
ahead and tag methods with required anyway.  Instead, if we see
this in something other than a protocol, just ignore the request.

Also, improve error recovery a bit when we see something bogus 
inside an interface.

Modified:
    cfe/trunk/lib/Parse/ParseObjc.cpp

Modified: cfe/trunk/lib/Parse/ParseObjc.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Parse/ParseObjc.cpp?rev=57798&r1=57797&r2=57798&view=diff

==============================================================================
--- cfe/trunk/lib/Parse/ParseObjc.cpp (original)
+++ cfe/trunk/lib/Parse/ParseObjc.cpp Mon Oct 20 00:57:40 2008
@@ -281,24 +281,40 @@
     
     // Otherwise, we have an @ directive, eat the @.
     SourceLocation AtLoc = ConsumeToken(); // the "@"
-    tok::ObjCKeywordKind ocKind = Tok.getObjCKeywordID();
+    tok::ObjCKeywordKind DirectiveKind = Tok.getObjCKeywordID();
     
-    if (ocKind == tok::objc_end) { // @end -> terminate list
+    if (DirectiveKind == tok::objc_end) { // @end -> terminate list
       AtEndLoc = AtLoc;
       break;
     } 
     
-    if (ocKind == tok::objc_required) { // protocols only
+    switch (DirectiveKind) {
+    default:
+      Diag(Tok, diag::err_objc_illegal_interface_qual);
+      ConsumeToken();
+      // Skip until we see an @ or } or ;
+      SkipUntil(tok::r_brace, tok::at);
+      break;
+      
+    case tok::objc_required:
       ConsumeToken();
-      MethodImplKind = ocKind;
+      // This is only valid on protocols.
       if (contextKey != tok::objc_protocol)
         Diag(AtLoc, diag::err_objc_protocol_required);
-    } else if (ocKind == tok::objc_optional) { // protocols only
+      else
+        MethodImplKind = tok::objc_required;
+      break;
+        
+    case tok::objc_optional:
       ConsumeToken();
-      MethodImplKind = ocKind;
+      // This is only valid on protocols.
       if (contextKey != tok::objc_protocol)
         Diag(AtLoc, diag::err_objc_protocol_optional);
-    } else if (ocKind == tok::objc_property) {
+      else
+        MethodImplKind = tok::objc_optional;
+      break;
+        
+    case tok::objc_property:
       ObjCDeclSpec OCDS;
       ConsumeToken(); // the "property" identifier
       // Parse property attribute list, if any. 
@@ -337,13 +353,12 @@
                              MethodImplKind);
         allProperties.push_back(Property);
       }
-      continue;
-    } else {
-      Diag(Tok, diag::err_objc_illegal_interface_qual);
-      ConsumeToken();
+      break;
     }
   }
-  /// Insert collected methods declarations into the @interface object.
+  // Insert collected methods declarations into the @interface object.
+  // FIXME: This passes in an invalid SourceLocation for AtEndLoc when EOF is
+  // hit.
   Actions.ActOnAtEnd(AtEndLoc, interfaceDecl,
                      allMethods.empty() ? 0 : &allMethods[0],
                      allMethods.size(),