cmd/ld: fix argument passing in doc.go

R=golang-dev, r
CC=golang-dev
https://codereview.appspot.com/8622043
encoding/json: different decision on tags and shadowing
If there are no tags, the rules are the same as before.
If there is a tagged field, choose it if there is exactly one
at the top level of all fields.
More tests. The old tests were clearly inadequate, since
they all pass as is. The new tests only work with the new code.

R=golang-dev, iant
CC=golang-dev
https://codereview.appspot.com/8617044
go/ast: distinguish between methods and functions in filtering

Go1.1 harmless, but not critical.

Fixes issue 5249.

R=golang-dev, r
CC=golang-dev
https://codereview.appspot.com/8609043
fmt: tweak the explanation of Stringer to be more correct and less specific
The String method is called whenever the printing operation wants a string,
not just for %s and %v.

R=golang-dev, bradfitz
CC=golang-dev
https://codereview.appspot.com/8638043
cmd/ld: fix scope of static objects in symbol table

All symbols are currently marked with global scope.
The correct behavior was lost when the symbol table
format was changed.

R=golang-dev, iant, r
CC=golang-dev
https://codereview.appspot.com/8625043
bufio/Scan: fix error handling at EOF
Fixes issue 5268.

R=golang-dev, dsymonds, bradfitz
CC=golang-dev
https://codereview.appspot.com/8646045
    cmd/cgo, cmd/go: remove #cgo directive parsing from cmd/cgo

This change removes processing of #cgo directives from cmd/cgo,
pushing the onus back on cmd/go to pass all necessary flags.

Fixes issue 5224. See comments for rationale.

R=golang-dev, iant, r
CC=golang-dev
https://codereview.appspot.com/8610044

Committer: Rob Pike <r@golang.org>
spec: add another sentence about BOMs
Although one may argue that they should be legal, gc (at least)
disallows byte order marks that are not the first code point
in the file. Added a sentence to the "Implementation restriction"
clause in the "Source code representation" section to document
this better.

Lifting this restriction (again - the rule has changed at least
twice already) would not break any existing programs, should
we later decide yet again to fiddle the rules about these little
fly specks.

R=golang-dev, dsymonds, gri
CC=golang-dev
https://codereview.appspot.com/8649043
gc: escape unicode BOM in exported string literals

Fixes issue 5260.

R=golang-dev, minux.ma, 0xjnml, r
CC=golang-dev
https://codereview.appspot.com/8658043

Committer: Rob Pike <r@golang.org>
encoding/json: documentation regarding anonymous fields.

R=golang-dev, iant
CC=golang-dev
https://codereview.appspot.com/8545046
misc/dist/darwin: add execute bit to preinstall
Fixes issue 5258.

R=golang-dev, iant, minux.ma
CC=golang-dev
https://codereview.appspot.com/8682043
runtime: use UMTX_OP_WAIT_UINT on FreeBSD

UMTX_OP_WAIT expects that the address points to a uintptr, but
the code in lock_futex.c uses a uint32.  UMTX_OP_WAIT_UINT is
just like UMTX_OP_WAIT, but the address points to a uint32.
This almost certainly makes no difference on a little-endian
system, but since the kernel supports it we should do the
right thing.  And, who knows, maybe it matters.

R=golang-dev, bradfitz, r, ality
CC=golang-dev
https://codereview.appspot.com/8699043
runtime: add a hook to disable precise GC

This will let us ask people to rebuild the Go system without
precise GC, and then rebuild and retest their program, to see
if precise GC is causing whatever problem they are having.

R=golang-dev, r
CC=golang-dev
https://codereview.appspot.com/8700043
cmd/ld: if we fail from an unexpected PE flags value, print it

R=golang-dev, minux.ma, r
CC=golang-dev
https://codereview.appspot.com/8709043
cmd/ld: always do external link for -linkmode=external

There are tests in run.bash for -linkmode=external.

Fixes issue 5238.

R=golang-dev, bradfitz, remyoudompheng, r
CC=golang-dev
https://codereview.appspot.com/8716044
cmd/go: quote command line arguments in debug output

Debug output from go test -x may contain empty arguments.
This CL quotes arguments if needed. E.g. the output of
go test -x is now
  .../6g -o ./_go_.6 -p testmain -complete -D "" -I . -I $WORK ./_testmain.go
which is easier to grasp.

R=golang-dev, bradfitz, minux.ma, r
CC=golang-dev
https://codereview.appspot.com/8633043

Committer: Rob Pike <r@golang.org>
cmd/go: quote command line arguments in debug output

Debug output from go test -x may contain empty arguments.
This CL quotes arguments if needed. E.g. the output of
go test -x is now
  .../6g -o ./_go_.6 -p testmain -complete -D "" -I . -I $WORK ./_testmain.go
which is easier to grasp.

R=golang-dev, bradfitz, minux.ma, r
CC=golang-dev
https://codereview.appspot.com/8633043

Committer: Rob Pike <r@golang.org>
test: raise the allocation threshold for chan/select2.go failure

Updates issue 5282

R=golang-dev, r
CC=golang-dev
https://codereview.appspot.com/8718045
test: make goprint.go wait longer for go its routine to execute

Update issue 5281

R=golang-dev, r, bradfitz, cshapiro
CC=golang-dev
https://codereview.appspot.com/8631047
testing/iotest: rephrase DataErrReader's description

R=r
CC=golang-dev
https://codereview.appspot.com/8650044

Committer: Rob Pike <r@golang.org>