From 79f14789d803872a3ad7069c5046172050ce79fe Mon Sep 17 00:00:00 2001 From: Chris Lattner Date: Mon, 21 Jul 2008 05:54:02 +0000 Subject: tighten up some checks, don't allow sending a message to NSString**** llvm-svn: 53836 --- clang/lib/Sema/SemaExprObjC.cpp | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) (limited to 'clang/lib/Sema/SemaExprObjC.cpp') diff --git a/clang/lib/Sema/SemaExprObjC.cpp b/clang/lib/Sema/SemaExprObjC.cpp index 1067d1fb808..197bf372c4f 100644 --- a/clang/lib/Sema/SemaExprObjC.cpp +++ b/clang/lib/Sema/SemaExprObjC.cpp @@ -269,12 +269,13 @@ Sema::ExprResult Sema::ActOnInstanceMessage( return true; } } else { - bool receiverIsQualId = isa(receiverType); - // FIXME (snaroff): checking in this code from Patrick. Needs to be - // revisited. how do we get the ClassDecl from the receiver expression? - if (!receiverIsQualId) - while (const PointerType *PTy = receiverType->getAsPointerType()) + // We allow sending a message to a qualified ID ("id") to an interface + // directly ("[NSNumber foo]") and to a pointer to an interface (an object). + if (!isa(receiverType) && + !isa(receiverType)) + if (const PointerType *PTy = receiverType->getAsPointerType()) receiverType = PTy->getPointeeType(); + // else error, invalid receiver. ObjCInterfaceDecl* ClassDecl = 0; if (ObjCQualifiedIdType *QIT = -- cgit v1.2.3