code review 7593050: spec: fix description of initialization

spec: fix description of initialization
The analysis does not depend on the values of the items.
Fixes issue 4648.

[r, 2013-03-20 20:53:54]

Hello golang-dev@googlegroups.com,

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

[r, 2013-03-20 20:54:36]

This brings the spec in line with both the intent and the existing
implementations. The mention of values in the existing spec is a
mistake.

-rob

[gri, 2013-03-20 21:10:06]

+1 for removing the notion of value

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

https://codereview.appspot.com/7593050/diff/1/doc/go_spec.html#newcode5605
doc/go_spec.html:5605: topological reference order: if the initializer of
<code>A</code>
Is "topological reference order" actually an understood term?

how about:

... based on the order they are referenced: if the initializer....

https://codereview.appspot.com/7593050/diff/1/doc/go_spec.html#newcode5619
doc/go_spec.html:5619: in the order they appear in the source.
There may be multiple files involved, so the order also depends on the order in
which the files are presented to the compiler. This should be mentioned
somewhere (it's also part of the issue).

[r, 2013-03-20 21:24:54]

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

Please take another look.

[gri, 2013-03-20 21:28:22]

LGTM

https://codereview.appspot.com/7593050/diff/6001/doc/go_spec.html
File doc/go_spec.html (right):

https://codereview.appspot.com/7593050/diff/6001/doc/go_spec.html#newcode5719
doc/go_spec.html:5719: order of reference: if the initializer of <code>A</code>
I suggested "based on the order of reference" instead of "in the order of
reference" because the order is (topologically) reversed: If A depends on B, B
will be initialized first.

Maybe it's fine since there is an example.

[r, 2013-03-20 21:51:13]

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

Please take another look.

[rsc, 2013-03-20 22:01:47]

LGTM

[gri, 2013-03-20 22:02:54]

LGTM

[r, 2013-03-20 22:05:30]

*** Submitted as https://code.google.com/p/go/source/detail?r=85b1cc5d35dd ***

spec: fix description of initialization
The analysis does not depend on the values of the items.
Fixes issue 4648.

R=golang-dev, gri, rsc
CC=golang-dev
https://codereview.appspot.com/7593050