]> granicus.if.org Git - clang/commitdiff
Move FIXME to a better location.
authorSteve Naroff <snaroff@apple.com>
Thu, 8 Jan 2009 20:17:34 +0000 (20:17 +0000)
committerSteve Naroff <snaroff@apple.com>
Thu, 8 Jan 2009 20:17:34 +0000 (20:17 +0000)
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@61937 91177308-0d34-0410-b5e6-96231b3b80d8

lib/Sema/SemaDeclObjC.cpp

index 9fb69cb95b10aa7319d9683cd5d89d44444fdaf9..7623c08e152c8e8af65d3985d140856d167d9a8d 100644 (file)
@@ -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);