diff options
author | Steve Naroff <snaroff@apple.com> | 2009-01-08 20:17:34 +0000 |
---|---|---|
committer | Steve Naroff <snaroff@apple.com> | 2009-01-08 20:17:34 +0000 |
commit | 4fb78c6d62a325f949e002a6bb3af4f6f33608a9 (patch) | |
tree | 0ab894be2d71e5c722f7689d8da0ebc1280754af /lib/Sema/SemaDeclObjC.cpp | |
parent | 92f863bffa9d5efe5cc63992e06b268231bc2cc0 (diff) |
Move FIXME to a better location.
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@61937 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/Sema/SemaDeclObjC.cpp')
-rw-r--r-- | lib/Sema/SemaDeclObjC.cpp | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/lib/Sema/SemaDeclObjC.cpp b/lib/Sema/SemaDeclObjC.cpp index 9fb69cb95b..7623c08e15 100644 --- a/lib/Sema/SemaDeclObjC.cpp +++ b/lib/Sema/SemaDeclObjC.cpp @@ -1038,6 +1038,11 @@ void Sema::ProcessPropertyDecl(ObjCPropertyDecl *property, // Synthesize getter/setter methods if none exist. // Find the default getter and if one not found, add one. + // FIXME: The synthesized property we set here is misleading. We + // almost always synthesize these methods unless the user explicitly + // provided prototypes (which is odd, but allowed). Sema should be + // typechecking that the declarations jive in that situation (which + // it is not currently). if (!GetterMethod) { // No instance method of same name as property getter name was found. // Declare a getter method and add it to the list of methods @@ -1097,11 +1102,6 @@ void Sema::ProcessPropertyDecl(ObjCPropertyDecl *property, // double bar = [foo bar]; // } // - // FIXME: The synthesized property we set here is misleading. We - // almost always synthesize these methods unless the user explicitly - // provided prototypes (which is odd, but allowed). Sema should be - // typechecking that the declarations jive in that situation (which - // it is not currently). if (GetterMethod) { CD->addDecl(Context, GetterMethod); AddInstanceMethodToGlobalPool(GetterMethod); |