[cfe-commits] r56770 - in /cfe/trunk: lib/Sema/SemaExpr.cpp test/Sema/block-args.c

[steve naroff]

On Sep 28, 2008, at 7:06 PM, Chris Lattner wrote:

>
> On Sep 28, 2008, at 2:08 PM, Steve Naroff wrote:
>
>> Agreed. I just added a FIXME to the code. I can likely clean this  
>> up tomorrow.
>
> Thanks Steve!  One minor correction:
>
> +/// int main(int argc) {
> +///   int xx;
> +///   ^(int X) {
> +///     xx = 4; // error (variable is not assignable)
> +///     argc = 3; // no error.
>
> Assigning to argc (and xx) would be an error: assigning to X would  
> be ok.
>

Understood. The comment is documenting what clang does now. The  
preceding comment says "make sure both assignments below produce an  
error".

snaroff

> -Chris