R=golang-codereviews, dave, gobot, bradfitz, gri
CC=golang-codereviews
https://golang.org/cl/46640043

R=golang-codereviews, r, aram
CC=0intro, golang-codereviews, rsc
https://golang.org/cl/43420045

No change to the meaning, just bad writing found by
Doug McIlroy.
Let's start the new year off with a bang.

R=golang-codereviews, bradfitz, dave
CC=golang-codereviews
https://golang.org/cl/47110044

when comparing kinds.

R=dvyukov, dave, khr
CC=golang-codereviews
https://golang.org/cl/41660045

Fixes #7003.

R=golang-codereviews, bradfitz
CC=golang-codereviews
https://golang.org/cl/47780043

R=shawn.p.smith, gobot, r
CC=golang-codereviews
https://golang.org/cl/46820043

Fixes #7043.

Test coming in https://golang.org/cl/46820043

R=r, bradfitz
CC=golang-codereviews
https://golang.org/cl/38330045

Fixes #6406.

R=golang-codereviews, bradfitz
CC=golang-codereviews
https://golang.org/cl/46900043

encoding/json: Fix missing error when trying to unmarshal null string into int, for successive ,string option

Fixes #7046.

R=golang-codereviews, bradfitz
CC=golang-codereviews
https://golang.org/cl/47260043

Most BSDs include the trailing NUL character of the socket path in the
length, however some do not (such as NetBSD 6.99). Handle this by only
subtracting the family and length bytes from the returned length, then
scanning the path and removing any terminating NUL bytes.

Fixes #6627.

R=golang-codereviews, mikioh.mikioh
CC=golang-codereviews
https://golang.org/cl/46420044

Some builders broke on this test; I'm guessing that was because
this test didn't try hard enough to find a different iteration order.

Update #6719

R=dave
CC=golang-codereviews
https://golang.org/cl/47300043

Technically the spec does not guarantee that the iteration order is random,
but it is a property that we have consciously pursued, and so it seems
right to verify that our implementation does indeed randomise.

Update #6719.

R=khr, bradfitz
CC=golang-codereviews
https://golang.org/cl/47010043

Generated by addca.

R=gobot
CC=golang-codereviews
https://golang.org/cl/39780047

R=golang-codereviews, bradfitz
CC=golang-codereviews
https://golang.org/cl/46880043

R=golang-codereviews, rsc, dave
CC=golang-codereviews
https://golang.org/cl/42440043

R=golang-codereviews, shawn.p.smith
CC=golang-codereviews
https://golang.org/cl/41960047

R=golang-codereviews, dave
CC=golang-codereviews
https://golang.org/cl/42990045

R=golang-codereviews, dave, bradfitz
CC=golang-codereviews
https://golang.org/cl/46250043

R=golang-codereviews, dave
CC=golang-codereviews
https://golang.org/cl/46450043

R=golang-codereviews, dave
CC=golang-codereviews
https://golang.org/cl/46270043

This source file, when compiled with gcc 4.4.3 on Ubuntu lucid,
corresponds instruction for instruction to the binaries in the same
directory.

Shipping this source code file resolves http://bugs.debian.org/716853

R=golang-codereviews, iant
CC=golang-codereviews
https://golang.org/cl/46780043

R=golang-codereviews, dave
CC=golang-codereviews
https://golang.org/cl/46470043

R=adg, djd, mpvl, nigeltao
CC=golang-codereviews
https://golang.org/cl/46660043

R=golang-codereviews, dave
CC=golang-codereviews
https://golang.org/cl/46400044

R=golang-codereviews, dave
CC=golang-codereviews
https://golang.org/cl/46630044

R=golang-codereviews, bradfitz, dave
CC=golang-codereviews
https://golang.org/cl/46600043

R=golang-codereviews, bradfitz, dave
CC=golang-codereviews
https://golang.org/cl/43550047

Fixes #7032.

R=golang-codereviews, r
CC=golang-codereviews
https://golang.org/cl/43960045

Looks like a typo.
Fixes #7011.

R=golang-codereviews, r, bradfitz
CC=golang-codereviews
https://golang.org/cl/45350043

Generated by addca.

R=gobot
CC=golang-codereviews
https://golang.org/cl/46540044

This change fixes a serious performance regression
with reflect.Value growing to 4 words instead of 3.
The json benchmark was ~50% slower, with this change
it is ~5% slower (and the binary is 0.5% larger).

Longer term, we probably need to rethink our copy
generation.  Using REP is really expensive time-wise.
But inlining the copy grows the binary.

R=golang-codereviews, r
CC=golang-codereviews
https://golang.org/cl/44990043

on map keys and values which are now passed by reference.

R=dvyukov, khr
CC=golang-codereviews
https://golang.org/cl/43490044

R=golang-codereviews, bradfitz
CC=golang-codereviews
https://golang.org/cl/46500043

R=golang-codereviews, bradfitz
CC=golang-codereviews
https://golang.org/cl/46510043

R=golang-codereviews, iant
CC=golang-codereviews
https://golang.org/cl/46320043

R=golang-codereviews, iant
CC=golang-codereviews
https://golang.org/cl/39910044

R=golang-codereviews, dave
CC=golang-codereviews
https://golang.org/cl/39200045

R=golang-codereviews, dave
CC=golang-codereviews
https://golang.org/cl/41320044

Fixes #6952.

runtime.asminit was incorrectly loading runtime.goarm as a word, not a uint8 which made it subject to alignment issues on arm5 platforms.

Alignment aside, this also meant that the top 3 bytes in R11 would have been garbage and could not be assumed to be setting up the FPU reliably.

R=iant, minux.ma
CC=golang-codereviews
https://golang.org/cl/46240043

On Solaris, if you do a in-progress connect, and then the
server accepts and closes the socket, the client's later
attempt to complete the connect will fail with EINVAL.  Handle
this case by assuming that the connect succeeded.  This code
is weird enough that it is implemented as Solaris-only so that
it doesn't hide a real error on a different OS.

Update #6828

R=golang-codereviews, bradfitz, dave
CC=golang-codereviews
https://golang.org/cl/46160043