[cfe-commits] r65562 - in /cfe/trunk: lib/Sema/Sema.h lib/Sema/SemaDeclObjC.cpp test/SemaObjC/property-13.m

[Sebastian Redl]

Steve Naroff wrote:
> Author: snaroff
> Date: Thu Feb 26 13:11:32 2009
> New Revision: 65562
>
> URL: http://llvm.org/viewvc/llvm-project?rev=65562&view=rev
> Log:
> Fix <rdar://problem/6574319> clang issues error on 'readonly' property with a defaul setter attribute.
>
> Needed to make isPropertyReadonly() non-const (for this fix to compile). I imagine there's a way to retain the const-ness, however I have more important fish to fry.
>
> +  // Lastly, look through the implementation (if one is in scope).
> +  if (ObjCImplementationDecl *ImpDecl = 
> +        ObjCImplementations[IDecl->getIdentifier()])
> +    if (ImpDecl->getInstanceMethod(PDecl->getSetterName()))
> +      return false;
>    return true;
>  }
>   
Wouldn't it suffice to make ImpDecl here a pointer-to-const?

Sebastian