code review 5400045: gofmt: do not stop test.sh after 1 error

gofmt: do not stop test.sh after 1 error

Fix bug377.go to be gofmt-compliant.

[rsc, 2011-11-16 22:54:45]

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

I'd like you to review this change to
https://go.googlecode.com/hg

[rsc, 2011-11-16 22:55:12]

First palindromic Go CL number?

[r2, 2011-11-16 23:06:16]

i use a stricter definition: a palindrome must look right in a mirror, not just
by textual reversal. no one else does, though.

-rob

[r, 2011-11-16 23:09:03]

LGTM

although i thought of using a trap command to clean up, this is fine.

[gri, 2011-11-16 23:11:15]

http://codereview.appspot.com/5400045/diff/5001/test/fixedbugs/bug377.go
File test/fixedbugs/bug377.go (right):

http://codereview.appspot.com/5400045/diff/5001/test/fixedbugs/bug377.go#newc...
test/fixedbugs/bug377.go:9: package ignored
we have more of these elsewhere and I usually excluded those tests

[rsc, 2011-11-16 23:12:45]

On Wed, Nov 16, 2011 at 18:11,  <gri@golang.org> wrote:
> we have more of these elsewhere and I usually excluded those tests

For the files that need to have syntax errors and
the like, it's unavoidable.  But it's easy to avoid here.
That list is getting pretty long.

[gri, 2011-11-16 23:24:12]

LGTM

On Wed, Nov 16, 2011 at 3:12 PM, Russ Cox <rsc@golang.org> wrote:
> On Wed, Nov 16, 2011 at 18:11,  <gri@golang.org> wrote:
>> we have more of these elsewhere and I usually excluded those tests
>
> For the files that need to have syntax errors and
> the like, it's unavoidable.  But it's easy to avoid here.
> That list is getting pretty long.
>

[rsc, 2011-11-16 23:44:23]

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

gofmt: do not stop test.sh after 1 error

Fix bug377.go to be gofmt-compliant.

R=gri, r, r
CC=golang-dev
http://codereview.appspot.com/5400045