code review 5562051: FAQ: update to Go 1.

FAQ: update to Go 1.

[r, 2012-01-22 23:12:04]

Hello golang-dev@googlegroups.com,

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

[adg, 2012-01-22 23:16:57]

LGTM

http://codereview.appspot.com/5562051/diff/1/doc/go_faq.html
File doc/go_faq.html (right):

http://codereview.appspot.com/5562051/diff/1/doc/go_faq.html#newcode768
doc/go_faq.html:768: In Go 1, equality has been defined for structs and arrays,
so such
s/has been/is/ ?

[rsc, 2012-01-23 14:50:13]

LGTM

All of our other docs use tabs.  Should probably keep the tabs here too.

http://codereview.appspot.com/5562051/diff/4001/doc/go_faq.html
File doc/go_faq.html (right):

http://codereview.appspot.com/5562051/diff/4001/doc/go_faq.html#newcode193
doc/go_faq.html:193: production inside Google.  A public example is the server
behind
The doubled is in this sentence is odd.

http://codereview.appspot.com/5562051/diff/4001/doc/go_faq.html#newcode1351
doc/go_faq.html:1351: For instance, <a
href="/test/bench/shootout">test/bench/shootout</a>/pidigits depends on a
multi-precision math package, and the C
Unclear why you are linking to the directory instead of the file.
x3

[r2, 2012-01-23 16:27:24]

On Jan 23, 2012, at 6:50 AM, rsc@golang.org wrote:

> LGTM
> 
> All of our other docs use tabs.  Should probably keep the tabs here too.

That's not true. Effective Go, for instance, has no tabs and tmpltohtml takes
pains to turn tabs into spaces for guaranteed regular formatting.

> http://codereview.appspot.com/5562051/diff/4001/doc/go_faq.html
> File doc/go_faq.html (right):
> 
> http://codereview.appspot.com/5562051/diff/4001/doc/go_faq.html#newcode193
> doc/go_faq.html:193: production inside Google.  A public example is the
> server behind
> The doubled is in this sentence is odd.
> 
> http://codereview.appspot.com/5562051/diff/4001/doc/go_faq.html#newcode1351
> doc/go_faq.html:1351: For instance, <a
> href="/test/bench/shootout">test/bench/shootout</a>/pidigits depends on
> a multi-precision math package, and the C
> Unclear why you are linking to the directory instead of the file.
> x3
> 
> http://codereview.appspot.com/5562051/

[r, 2012-01-23 16:29:27]

http://codereview.appspot.com/5562051/diff/4001/doc/go_faq.html
File doc/go_faq.html (right):

http://codereview.appspot.com/5562051/diff/4001/doc/go_faq.html#newcode1351
doc/go_faq.html:1351: For instance, <a
href="/test/bench/shootout">test/bench/shootout</a>/pidigits depends on a
multi-precision math package, and the C
On 2012/01/23 14:50:13, rsc wrote:
> Unclear why you are linking to the directory instead of the file.
> x3

which file? there's a c version and multiple go versions. i could just remove
the link altogether, or link the prime go version.

[rsc, 2012-01-23 16:34:25]

We should figure out what the story is with tabs or not.
If you grep for '    ' (tab) in doc/*.html there are plenty of
tabs in, for example, the spec.

It seems like we should link to the code being discussed,
so the go file for the shootout that does the things being
talked about (uses math/big, etc).

Russ

[r2, 2012-01-23 16:37:15]

On Jan 23, 2012, at 8:34 AM, Russ Cox wrote:

> We should figure out what the story is with tabs or not.
> If you grep for '    ' (tab) in doc/*.html there are plenty of
> tabs in, for example, the spec.

ok, but not here.

> It seems like we should link to the code being discussed,
> so the go file for the shootout that does the things being
> talked about (uses math/big, etc).

well, that's exactly the issue: what's being discussed? i'll just link to the go
version. it's weird grammatically but it's offline so there's kinda no grammar
involved.

will fix and submit

-rob

[r, 2012-01-23 16:39:57]

*** Submitted as 21c56311deab ***

FAQ: update to Go 1.

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