From: Steve Naroff <snaroff@apple.com>
Date: Tue, 5 Feb 2008 23:59:27 +0000 (+0000)
Subject: Add a FIXME to clarify previous commit/experiment.
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=5e7dc44adec45c5c7dcd6b86172e59e9a1f10851;p=clang

Add a FIXME to clarify previous commit/experiment.



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

diff --git a/Parse/ParseObjc.cpp b/Parse/ParseObjc.cpp
index 890b9bcaf9..01b2abffb0 100644
--- a/Parse/ParseObjc.cpp
+++ b/Parse/ParseObjc.cpp
@@ -1267,6 +1267,12 @@ Parser::StmtResult Parser::ParseObjCAtStatement(SourceLocation AtLoc) {
     bool parsedAtSign;
     
     StmtResult Res = ParseObjCTryStmt(AtLoc, parsedAtSign);
+    // FIXME: This hack results in a dropped AST node. To correctly implement 
+    // the hack, parseAtSign would need to bubble up to 
+    // ParseCompoundStatement(). This would involve adding an argument to this 
+    // routine and ParseStatementOrDeclaration(). Changing the parser in this
+    // fashion to solve such a conceptually simple problem is undesirable.
+    // Rework this clause once 2-token lookahead is implemented.
     if (!Res.isInvalid && parsedAtSign)
       return ParseObjCAtStatement(AtLoc);
     return Res;