bytes: don't compact Buffer so aggressively

benchmark                           old ns/op    new ns/op    delta
BenchmarkBufferNotEmptyWriteRead       848416       819983   -3.35%

Update Issue 5154

R=golang-dev, gri, robryk
CC=golang-dev
https://codereview.appspot.com/8173043
runtime: additional map benchmarks for repeated lookups

For the future.

Update Issue 5147

R=khr, r
CC=golang-dev
https://codereview.appspot.com/8165044
bytes: document that Buffer.Write grows the buffer
Do a little reformatting too.
Fixes issue 5152.

R=golang-dev, bradfitz, gri
CC=golang-dev
https://codereview.appspot.com/8157044
A+C: Robert Obryk (individual CLA)

Generated by addca.

R=gobot
CC=golang-dev
https://codereview.appspot.com/8176043
bytes: Benchmark Buffer's compactions

This benchmark verifies that CL #8173043 reduces time spent
sliding the Buffer's contents.

Results without and with CL #8173043 applied:
benchmark                        old ns/op    new ns/op    delta
BenchmarkBufferFullSmallReads       755336       175054  -76.82%

R=golang-dev, bradfitz
CC=golang-dev
https://codereview.appspot.com/8174043

Committer: Brad Fitzpatrick <bradfitz@golang.org>
cmd/dist, cmd/ld: GO_EXTLINK_ENABLED=0 defaults to -linkmode=internal

Change build system to set GO_EXTLINK_ENABLED=0 by default for
OS X 10.6, since the system linker has a bug and can not
handle the object files generated by 6l.

Fixes issue 5130.

R=golang-dev, r
CC=golang-dev
https://codereview.appspot.com/8183043
net/http: ignore 100-continue responses in Transport

"There are only two hard problems in computer science:
cache invalidation, naming things, and off-by-one errors."

The HTTP server code already strips Expect: 100-continue on
requests, so httputil.ReverseProxy should be unaffected, but
some servers send unsolicited HTTP/1.1 100 Continue responses,
so we need to skip over them if they're seen to avoid getting
off-by-one on Transport requests/responses.

This does change the behavior of people who were using Client
or Transport directly and explicitly setting "Expect: 100-continue"
themselves, but it didn't work before anyway. Now instead of the
user code seeing a 100 response and then things blowing up, now
it basically works, except the Transport will still blast away
the full request body immediately.  That's the part that needs
to be finished to close this issue.

This is the safe quick fix.

Update Issue 3665

R=golang-dev, dsymonds, dave, jgrahamc
CC=golang-dev
https://codereview.appspot.com/8166045
misc/dashboard/codereview: add Rémy to the list of assignees

R=dsymonds, mikioh.mikioh
CC=golang-dev
https://codereview.appspot.com/8094047
net: avoid use of listener socket options on active open sockets

This CL ensures we use the correct socket options for
passive and active open sockets.

For the passive open sockets created by Listen functions,
additional SO_REUSEADDR, SO_REUSEPORT options are required
for the quick service restart and/or multicasting.

For the active open sockets created by Dial functions, no
additional options are required.

R=golang-dev, dave, bradfitz, rsc
CC=golang-dev
https://codereview.appspot.com/7795050
misc/cgo/gmp: fix typo.

R=golang-dev, minux
CC=golang-dev
https://codereview.appspot.com/8174045

Committer: Shenghou Ma <minux.ma@gmail.com>
include/plan9/libc.h: fix Plan 9 build

Add missing getgoextlinkenabled(void) declaration
in Plan 9 libc.h. This function was added as part
of CL #8183043.

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

Committer: Shenghou Ma <minux.ma@gmail.com>
os/exec, syscall: fix tests to pass when GOGCTRACE=1 is set

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

Committer: Brad Fitzpatrick <bradfitz@golang.org>
cmd/6g: fix warnings on Plan 9

src/cmd/6g/peep.c:471 set and not used: r
src/cmd/6g/peep.c:560 overspecified class: regconsttyp GLOBL STATIC
src/cmd/6g/peep.c:761 more arguments than format IND STRUCT Prog
src/cmd/6g/reg.c:185 set and not used: r1
src/cmd/6g/reg.c:786 format mismatch d VLONG, arg 3
src/cmd/6g/reg.c:1064 format mismatch d VLONG, arg 5

R=golang-dev, bradfitz
CC=golang-dev
https://codereview.appspot.com/8197044

Committer: Brad Fitzpatrick <bradfitz@golang.org>
cmd/gc, cmd/ld: fix warnings on Plan 9

src/cmd/gc/closure.c:133 param declared and not used: nowrap
src/cmd/gc/const.c:1139 set and not used: t1
src/cmd/ld/data.c:652 format mismatch #llx INT, arg 7
src/cmd/ld/data.c:652 format mismatch #llx INT, arg 8
src/cmd/ld/data.c:1230 set and not used: datsize

R=dave, golang-dev, lucio.dere, remyoudompheng, bradfitz
CC=golang-dev
https://codereview.appspot.com/8182043

Committer: Brad Fitzpatrick <bradfitz@golang.org>
misc/emacs: Enable compilation-mode for gofmt error buffer before displaying it.

Some packages, like popwin.el, change display behaviour based on
the buffer's mode, so we should enable compilation-mode before
displaying the buffer.

R=golang-dev, bradfitz
CC=golang-dev
https://codereview.appspot.com/8155043

Committer: Brad Fitzpatrick <bradfitz@golang.org>
net/textproto: report allocs in benchmarks

R=golang-dev, minux.ma
CC=golang-dev
https://codereview.appspot.com/8187045
doc/go1.1: mention (*testing.B).ReportAllocs()

R=golang-dev, r
CC=golang-dev
https://codereview.appspot.com/8198043
net/http: fix incredibly racy TestTransportReading100Continue

Whoops. I'm surprised it even worked before. (Need two pipes,
not one.)

Also, remove the whole pipe registration business, since it
wasn't even required in the previous version. (I'd later fixed
it at the end of send100Response, but forgot to delete it)

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