[cfe-commits] r124668 - in /cfe/trunk: include/clang/Basic/DiagnosticSemaKinds.td include/clang/Sema/Sema.h lib/Sema/SemaExpr.cpp test/Analysis/self-init.m test/SemaCXX/warn-assignment-condition.cpp

Argyrios Kyrtzidis kyrtzidis at apple.com
Tue Feb 1 14:12:16 PST 2011


On Feb 1, 2011, at 2:09 PM, jahanian wrote:

> 
> On Feb 1, 2011, at 2:04 PM, Argyrios Kyrtzidis wrote:
>>> 
>> 
>> Why not,
>> 
>> if (((do-something-elaborate-that-results-in-lvalue-that-i-want-to-assign-and-check) = b))
> 
> It is subjective of course. I thought, there should be a way to escape this warning (other than by flag), and putting paren 
> is a way to make the intention known.

One can always remove the extra parens that serve no discernible purspose :-)

> 
> - Fariborz
> 
>> 
>> seems reasonable to me.
>> 
>>> 
>>> - Fariborz
>>> 
>>>> 
>>>> -Argiris
>>>> _______________________________________________
>>>> cfe-commits mailing list
>>>> cfe-commits at cs.uiuc.edu
>>>> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>>> 
>>> 
>>> _______________________________________________
>>> cfe-commits mailing list
>>> cfe-commits at cs.uiuc.edu
>>> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>> 
> 

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20110201/600d9fae/attachment.html>


More information about the cfe-commits mailing list