[cfe-commits] r60629 - /cfe/trunk/lib/Sema/SemaDeclObjC.cpp

[Chris Lattner]

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
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits