delta/go.git, branch dev.cc

[dev.cc] all: merge default (8d42099cdc23) into dev.cc

2014-12-05T16:18:10+00:00

TBR=austin
CC=golang-codereviews
https://codereview.appspot.com/178700044

[dev.cc] liblink: don't patch jumps to jumps to symbols

2014-12-05T14:24:01+00:00

When liblink sees something like

       JMP x
       ...
    x: JMP y

it rewrites the first jump to jump directly to y.  This is
fine if y is a resolved label.  However, it *also* does this
if y is a function symbol, but fails to carry over the
relocation that would later patch in that symbol's value.  As
a result, the original jump becomes either a self-jump (if
relative) or a jump to PC 0 (if absolute).

Fix this by disabling this optimization if the jump being
patched in is a jump to a symbol.

LGTM=minux
R=rsc, minux
CC=golang-codereviews
https://codereview.appspot.com/185890044

[dev.cc] cmd/ld: finalize linkmode before determining whether to import runtime/cgo

2014-12-05T07:22:20+00:00

Frankly, I don't understand how the current code could possibly work except
when every android program is using cgo. Discovered this while working on
the iOS port.

LGTM=crawshaw, rsc
R=rsc, crawshaw
CC=golang-codereviews
https://codereview.appspot.com/177470043

cmd/go: fix build

2014-12-05T00:37:56+00:00

The new semantics of split require the newline be present.
The test was stale.

LGTM=adg
R=golang-codereviews, adg
CC=golang-codereviews
https://codereview.appspot.com/182480043

cmd/go: avoid use of bufio.Scanner in generate

2014-12-05T00:15:38+00:00

Scanner can't handle stupid long lines and there are
reports of stupid long lines in production.

Note the issue isn't long "//go:generate" lines, but
any long line in any Go source file.

To be fair, if you're going to have a stupid long line
it's not a bad bet you'll want to run it through go
generate, because it's some embeddable asset that
has been machine generated. (One could ask why
that generation process didn't add a newline or two,
but we should cope anyway.)

Rewrite the file scanner in "go generate" so it can
handle arbitrarily long lines, and only stores in memory
those lines that start "//go:generate".

Also: Adjust the documentation to make clear that it
does not parse the file.

Fixes issue 9143.
Fixes issue 9196.

LGTM=rsc, dominik.honnef
R=rsc, cespare, minux, dominik.honnef
CC=golang-codereviews
https://codereview.appspot.com/182970043

cmd/pprof/internal/commands: add command to open browser on windows

2014-12-04T16:24:23+00:00

While we're at there, also add a message to prompt the user to install
Graphviz if "dot" command is not found.

Fixes issue 9178.

LGTM=adg, alex.brainman, cookieo9, rsc
R=rsc, adg, bradfitz, alex.brainman, cookieo9, smyrman
CC=golang-codereviews
https://codereview.appspot.com/180380043

Committer: Russ Cox

lib/time: update to ICANN time zone database 2014j

2014-12-04T04:07:48+00:00

Fixes issue 9189.

LGTM=dsymonds
R=golang-codereviews, dsymonds
CC=golang-codereviews
https://codereview.appspot.com/178660043

spec: add comment marker for consistency.

2014-12-03T22:29:29+00:00

LGTM=r
R=gri, r
CC=golang-codereviews
https://codereview.appspot.com/185830043

cmd/pprof: fix symbol resolution for remote profiles

2014-12-03T19:14:00+00:00

Fixes issue 9199.

LGTM=iant
R=golang-codereviews, iant
CC=austin, golang-codereviews, minux
https://codereview.appspot.com/183080043

cmd/go: regenerate doc.go

2014-12-02T23:28:54+00:00

Move change from CL 170770043 to correct file and regenerate docs
for changes from CL 164120043.

LGTM=adg
R=golang-codereviews, adg, bradfitz
CC=golang-codereviews
https://codereview.appspot.com/183000043

Committer: Andrew Gerrand