From 9a207ee4dc85e08be4935f3f972e066314f680e2 Mon Sep 17 00:00:00 2001 From: Fariborz Jahanian Date: Tue, 25 Nov 2008 17:56:43 +0000 Subject: Patch to allow over-riding of readonly property to a writable property in one of its category. llvm-svn: 60035 --- clang/lib/AST/Expr.cpp | 22 ++++++++++++++++++---- 1 file changed, 18 insertions(+), 4 deletions(-) (limited to 'clang/lib/AST') diff --git a/clang/lib/AST/Expr.cpp b/clang/lib/AST/Expr.cpp index 835a467fc21..56f08b28d95 100644 --- a/clang/lib/AST/Expr.cpp +++ b/clang/lib/AST/Expr.cpp @@ -551,14 +551,28 @@ Expr::isModifiableLvalueResult Expr::isModifiableLvalue(ASTContext &Ctx) const { if (getStmtClass() == ObjCPropertyRefExprClass) { const ObjCPropertyRefExpr* PropExpr = cast(this); if (ObjCPropertyDecl *PDecl = PropExpr->getProperty()) { - ObjCPropertyDecl::PropertyAttributeKind Pkind = - PDecl->getPropertyAttributes(); - if (Pkind == ObjCPropertyDecl::OBJC_PR_readonly) + if (PDecl->isReadOnly()) { + // Main class has the property as 'readyonly'. Must search + // through the category list to see if the property's + // attribute has been over-ridden to 'readwrite'. + const Expr *BaseExpr = PropExpr->getBase(); + QualType BaseType = BaseExpr->getType(); + const PointerType *PTy = BaseType->getAsPointerType(); + const ObjCInterfaceType *IFTy = + PTy->getPointeeType()->getAsObjCInterfaceType(); + ObjCInterfaceDecl *IFace = IFTy->getDecl(); + for (ObjCCategoryDecl *Category = IFace->getCategoryList(); + Category; Category = Category->getNextClassCategory()) { + PDecl= Category->FindPropertyDeclaration(PDecl->getIdentifier()); + if (PDecl && !PDecl->isReadOnly()) + return MLV_Valid; + } return MLV_ReadonlyProperty; + } } } // Assigning to an 'implicit' property? - if (getStmtClass() == ObjCKVCRefExprClass) { + else if (getStmtClass() == ObjCKVCRefExprClass) { const ObjCKVCRefExpr* KVCExpr = cast(this); if (KVCExpr->getSetterMethod() == 0) return MLV_NoSetterProperty; -- cgit v1.2.3