From 4fb78c6d62a325f949e002a6bb3af4f6f33608a9 Mon Sep 17 00:00:00 2001 From: Steve Naroff Date: Thu, 8 Jan 2009 20:17:34 +0000 Subject: [PATCH] Move FIXME to a better location. git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@61937 91177308-0d34-0410-b5e6-96231b3b80d8 --- lib/Sema/SemaDeclObjC.cpp | 10 +++++----- 1 file 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); -- 2.40.0