code review 869041: test for panic and recover

test for panic and recover

[rsc, 2010-03-31 08:46:31]

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

I'd like you to review this change.

[adg1, 2010-03-31 11:15:48]

LGTM.

Fun bit of code =)

On 31 March 2010 19:46,  <rsc@golang.org> wrote:
> Reviewers: golang-dev_googlegroups.com,
>
> Message:
> Hello golang-dev@googlegroups.com (cc: golang-dev@googlegroups.com),
>
> I'd like you to review this change.
>
>
> Description:
> test for panic and recover
>
> Please review this at http://codereview.appspot.com/869041/show
>
> Affected files:
>  A test/recover.go
>  A test/recover1.go
>
>
>

[r, 2010-03-31 16:01:31]

LGTM

http://codereview.appspot.com/869041/diff/11001/12001
File test/recover.go (right):

http://codereview.appspot.com/869041/diff/11001/12001#newcode123
test/recover.go:123: defer func() {
i'd like another version where instead of returning deflt it's
x = deflt
defer ...
return

[rsc, 2010-03-31 17:08:18]

done

[rsc, 2010-03-31 18:47:13]

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

test for panic and recover

R=r, adg
CC=golang-dev
http://codereview.appspot.com/869041