[cfe-commits] r95838 - in /cfe/trunk: include/clang/AST/Expr.h include/clang/Basic/DiagnosticSemaKinds.td lib/AST/Expr.cpp lib/Sema/SemaExpr.cpp test/SemaObjC/property-not-lvalue.m

[Fariborz Jahanian]

On Feb 11, 2010, at 9:43 AM, Fariborz Jahanian wrote:

>
> On Feb 11, 2010, at 8:22 AM, Daniel Dunbar wrote:
>
>> Hi Fariborz,
>>
>> This doesn't feel right to me. Why is this different than:
>> --
>> struct s0 { int x; };
>> struct s0 f0();
>> void f1() { f0().x = 1; }
>
> Property syntax can always be converted into objective-c messaging
> syntax. So, in this
> case it is equivalent to [self size].width = 2.2; which issues an
> lvalue diagnostic in objective-c mode
> (as does above test case).
> Diagnostics should not be issued on objective-c++ mode, however. Will
> change this.
Done in r96012.
- fj
>
>
> - fariborz