On Dec 6, 2008, at 1:11 PM, Fariborz Jahanian wrote: > Author: fjahanian > Date: Sat Dec 6 15:11:37 2008 > New Revision: 60629 > > URL: http://llvm.org/viewvc/llvm-project?rev=60629&view=rev > Log: > Check for correct'void' return type for setter methods.
Hi Fariborz, does this work on methods that return a typedef for void? -Chris > > > Modified: > cfe/trunk/lib/Sema/SemaDeclObjC.cpp > > Modified: cfe/trunk/lib/Sema/SemaDeclObjC.cpp > URL: > http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Sema/SemaDeclObjC.cpp?rev=60629&r1=60628&r2=60629&view=diff > > = > = > = > = > = > = > = > = > ====================================================================== > --- cfe/trunk/lib/Sema/SemaDeclObjC.cpp (original) > +++ cfe/trunk/lib/Sema/SemaDeclObjC.cpp Sat Dec 6 15:11:37 2008 > @@ -940,7 +940,7 @@ > << GetterMethod->getSelector().getAsIdentifierInfo(); > > if (SetterMethod) { > - if (SetterMethod->getResultType() != Context.VoidPtrTy) > + if (SetterMethod->getResultType() != Context.VoidTy) > Diag(SetterMethod->getLocation(), diag::err_setter_type_void); > if (SetterMethod->getNumParams() != 1 || > (SetterMethod->getParamDecl(0)->getType() != property- > >getType())) > > > _______________________________________________ > cfe-commits mailing list > [email protected] > http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
