code review 6653047: spec: more clarification about deferred functions

spec: more clarification about deferred functions
Proposed new text to make matters clearer. The existing text was
unclear about the state of result parameters when panicking.

[r, 2012-10-12 04:00:11]

Hello golang-dev@googlegroups.com,

I'd like you to review this change to
https://code.google.com/p/go/

[iant, 2012-10-12 05:21:08]

https://codereview.appspot.com/6653047/diff/1/doc/go_spec.html
File doc/go_spec.html (right):

https://codereview.appspot.com/6653047/diff/1/doc/go_spec.html#newcode4760
doc/go_spec.html:4760: <a href="#Run_time_panics">run-time panic</a> occurs, as
described
The reference to the run-time panic section seems slightly misleading to me in
isolation.  That section is only about panics triggered by the runtime.  I think
that suggests that deferred functions are not run when the panic function is
called.

https://codereview.appspot.com/6653047/diff/1/doc/go_spec.html#newcode5080
doc/go_spec.html:5080: <code>F</code> returns to its caller with executing the
rest of
s/with/without/ ?

[r, 2012-10-12 05:24:49]

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

Please take another look.

[iant, 2012-10-12 05:54:22]

LGTM

https://codereview.appspot.com/6653047/diff/3002/doc/go_spec.html
File doc/go_spec.html (right):

https://codereview.appspot.com/6653047/diff/3002/doc/go_spec.html#newcode4738
doc/go_spec.html:4738: (See aslo the section on <a
href="#Handling_panics">handling panics</a>.)
s/aslo/also/

[r, 2012-10-14 23:39:24]

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

Please take another look.

[iant, 2012-10-15 16:43:43]

LGTM

[r, 2012-10-16 00:27:31]

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

spec: more clarification about deferred functions
Proposed new text to make matters clearer. The existing text was
unclear about the state of result parameters when panicking.

R=golang-dev, iant
CC=golang-dev
http://codereview.appspot.com/6653047

[gri, 2012-10-16 04:23:36]

LGTM