From: Steve Naroff <snaroff@apple.com>
Date: Wed, 22 Aug 2007 23:18:22 +0000 (+0000)
Subject: With this commit, we now successfully parse "Cocoa.h"!
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=4fa7afd07421e7276d1717e4fdf43a5fdd65a622;p=clang

With this commit, we now successfully parse "Cocoa.h"!



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

diff --git a/Parse/ParseDecl.cpp b/Parse/ParseDecl.cpp
index d4017bba1a..e5c5bb97b2 100644
--- a/Parse/ParseDecl.cpp
+++ b/Parse/ParseDecl.cpp
@@ -356,8 +356,7 @@ void Parser::ParseSpecifierQualifierList(DeclSpec &DS) {
 /// [GNU]   '_Decimal128'
 /// [GNU]   typeof-specifier
 /// [OBJC]  class-name objc-protocol-refs[opt]    [TODO]
-/// [OBJC]  typedef-name objc-protocol-refs       [TODO]
-/// [OBJC]  objc-protocol-refs                    [TODO]
+/// [OBJC]  typedef-name objc-protocol-refs[opt]  [TODO]
 ///       type-qualifier:
 ///         'const'
 ///         'volatile'
@@ -384,7 +383,15 @@ void Parser::ParseDeclarationSpecifiers(DeclSpec &DS) {
                                                CurScope)) {
           isInvalid = DS.SetTypeSpecType(DeclSpec::TST_typedef, Loc, PrevSpec,
                                          TypeRep);
-          break;
+          if (isInvalid)
+            break;
+          else { // FIXME: restrict this to "id" and ObjC classnames.
+            DS.Range.setEnd(Tok.getLocation());
+            ConsumeToken(); // The identifier
+            if (Tok.getKind() == tok::less)
+              ParseObjCProtocolReferences();
+            continue;
+          }
         }
       }
       // FALL THROUGH.
diff --git a/Parse/ParseObjc.cpp b/Parse/ParseObjc.cpp
index 4aaaecc757..4f4bdfe012 100644
--- a/Parse/ParseObjc.cpp
+++ b/Parse/ParseObjc.cpp
@@ -301,6 +301,20 @@ IdentifierInfo *Parser::ParseObjCSelector() {
   return II;
 }
 
+///   objc-type-qualifier: one of
+///     in out inout bycopy byref oneway
+///
+///   FIXME: remove the string compares...
+bool Parser::isObjCTypeQualifier() {
+  if (Tok.getKind() == tok::identifier) {
+    const char *qual = Tok.getIdentifierInfo()->getName();
+    return (strcmp(qual, "in") == 0) || (strcmp(qual, "out") == 0) ||
+           (strcmp(qual, "inout") == 0) || (strcmp(qual, "oneway") == 0) ||
+           (strcmp(qual, "bycopy") == 0) || (strcmp(qual, "byref") == 0);
+  }
+  return false;
+}
+
 ///   objc-type-name:
 ///     '(' objc-type-qualifiers[opt] type-name ')'
 ///     '(' objc-type-qualifiers[opt] ')'
@@ -309,14 +323,14 @@ IdentifierInfo *Parser::ParseObjCSelector() {
 ///     objc-type-qualifier
 ///     objc-type-qualifiers objc-type-qualifier
 ///
-///   objc-type-qualifier: one of
-///     in out inout bycopy byref oneway
-///
 void Parser::ParseObjCTypeName() {
   assert(Tok.getKind() == tok::l_paren && "expected (");
   
   SourceLocation LParenLoc = ConsumeParen(), RParenLoc;
   
+  while (isObjCTypeQualifier())
+    ConsumeToken();
+
   if (isTypeSpecifierQualifier()) {
     //TypeTy *Ty = ParseTypeName();
     //assert(Ty && "Parser::ParseObjCTypeName(): missing type");
diff --git a/include/clang/Parse/Parser.h b/include/clang/Parse/Parser.h
index 7ad9a0f659..90988791de 100644
--- a/include/clang/Parse/Parser.h
+++ b/include/clang/Parse/Parser.h
@@ -267,6 +267,7 @@ private:
   DeclTy *ParseObjCAtAliasDeclaration();
   
   IdentifierInfo *ParseObjCSelector();
+  bool isObjCTypeQualifier();
   void ParseObjCTypeName();
   void ParseObjCMethodRequirement();
   void ParseObjCMethodPrototype();