code review 179105: fix build, enable an exp/eval's assignment check

fix build, enable an exp/eval's assignment check
now that the parser doesn't do this test anymore

[gri, 2009-12-18 19:18:14]

Hello rsc (cc: golang-dev@googlegroups.com),

I'd like you to review the following change.

[rsc, 2009-12-18 19:21:00]

LGTM

On Fri, Dec 18, 2009 at 11:18,  <gri@golang.org> wrote:
> Reviewers: rsc,
>
> Message:
> Hello rsc (cc: golang-dev@googlegroups.com),
>
> I'd like you to review the following change.
>
>
> Description:
> fix build, enable an exp/eval's assignment check
> now that the parser doesn't do this test anymore
>
> Please review this at http://codereview.appspot.com/179105
>
> Affected files:
>  M src/pkg/exp/eval/stmt_test.go
>
>
> Index: src/pkg/exp/eval/stmt_test.go
> ===================================================================
> --- a/src/pkg/exp/eval/stmt_test.go
> +++ b/src/pkg/exp/eval/stmt_test.go
> @@ -19,11 +19,7 @@
>        Val2("a, b := 1, 2", "a", 1, "b", 2),
>        Val2("a, i := 1, 2", "a", 1, "i", 2),
>        CErr("a, i := 1, f", opTypes),
> -       // TODO(austin) The parser produces an error message for this
> -       // one that's inconsistent with the errors I give for other
> -       // things
> -       //CErr("a, b := 1, 2, 3", "too many"),
> -       CErr("a, b := 1, 2, 3", "arity"),
> +       CErr("a, b := 1, 2, 3", "too many"),
>        CErr("a := 1, 2", "too many"),
>        CErr("a, b := 1", "not enough"),
>        // Mixed declarations
>
>
>

[gri, 2009-12-18 19:21:16]

*** Submitted as http://code.google.com/p/go/source/detail?r=62648b222a7c ***

fix build, enable an exp/eval's assignment check
now that the parser doesn't do this test anymore

R=rsc
CC=golang-dev
http://codereview.appspot.com/179105