cmd/vet: don't give error for Printf("%+5.2e", x)

R=golang-dev, r
CC=golang-dev
https://golang.org/cl/5696048

doc: update codelab/wiki to Go 1.

R=golang-dev, r, adg
CC=golang-dev
https://golang.org/cl/5683076

os: fix minor typo

R=golang-dev, r
CC=golang-dev
https://golang.org/cl/5697051

test/bench/shootout: update post-Makefile

R=golang-dev, rsc
CC=golang-dev
https://golang.org/cl/5696054

reflect.DeepEqual: don't panic comparing functions
Functions are equal iff they are both nil.

Fixes #3122.

R=golang-dev, dsymonds, rsc
CC=golang-dev
https://golang.org/cl/5693057

test: document ken/*.go

R=golang-dev, rsc
CC=golang-dev
https://golang.org/cl/5694065

runtime: fix arm

signal.test binary passes on my phone; should fix builder

R=golang-dev, r
CC=golang-dev
https://golang.org/cl/5694064

gc: fix escape analysis + inlining + closure bug

R=ken2
CC=golang-dev, lvd
https://golang.org/cl/5693056

test: fix the fix of the rename tests.
Now they actually test again instead of just setting iota to zero.

R=golang-dev, rsc
CC=golang-dev
https://golang.org/cl/5700058

ld: fix alignment of rodata section

We were not aligning the code size,
so read-only data, which follows in the same
segment, could be arbitrarily misaligned.

Fixes #2506.

R=golang-dev, iant
CC=golang-dev
https://golang.org/cl/5693055

spec: allow disallow of \uFFFD in import path

Turns out U+FFFD is a graphic character too.

http://play.golang.org/p/YblCfKAfko

R=gri
CC=golang-dev
https://golang.org/cl/5699052

cmd/cc: grow some global arrays

Avoids global array buffer overflows if they are
indexed using some of the values between NTYPE
and NALLTYPE.  It is entirely likely that not all of these
are necessary, but this is the C compiler and not worth
worrying much about.  This change takes up only a
few more bytes of memory and makes the behavior
deterministic.

Fixes #3078.

R=golang-dev, bradfitz
CC=golang-dev
https://golang.org/cl/5693052

net/rpc: silence read error on closing connection

Fixes #3113.

R=golang-dev, r
CC=golang-dev
https://golang.org/cl/5698056

net: reorganize test files

R=golang-dev, rsc
CC=golang-dev
https://golang.org/cl/5694063

test: add cmpout to testlib

R=golang-dev, r
CC=golang-dev
https://golang.org/cl/5699060

doc/go1: document ProcessState
Also reformat the "go fix" references to make them look better by using the non-CW space.

Fixes #3087.

R=golang-dev, bradfitz
CC=golang-dev
https://golang.org/cl/5695057

test: add temporary show_skips flag.

To find test files with legacy headers.

We can remove this flag once everything is converted.

R=golang-dev, r
CC=golang-dev
https://golang.org/cl/5694060

godoc: fine-tuning .css

R=adg
CC=golang-dev
https://golang.org/cl/5699057

net/http: add overlooked 418 status code, per RFC 2324.

R=golang-dev, bradfitz, rsc
CC=golang-dev
https://golang.org/cl/5688067

test/[n-z]*.go: add documentation

R=golang-dev, bradfitz, r
CC=golang-dev
https://golang.org/cl/5700056

html/template: make doctype check case-insensitive

Fixes #3094.

R=golang-dev, rsc, nigeltao
CC=golang-dev
https://golang.org/cl/5687065

test/[n-r]*.go: add documentation
The rename ones needed redoing.

R=golang-dev, bradfitz, rsc
CC=golang-dev
https://golang.org/cl/5698054

godoc: bring back highlighting, selections, and alerts

R=adg, rsc
CC=golang-dev
https://golang.org/cl/5695054

godoc: consistent placement of documentation sections

Fixes #1226.

R=golang-dev, rsc
CC=golang-dev
https://golang.org/cl/5694052

runtime: check for ARM syscall failures
    While we are at it, also update some comments.
    Tested on Linux/ARM builder.

R=rsc, golang-dev
CC=golang-dev
https://golang.org/cl/5696047

dist: treat CC as one unit

Fixes #3112.

R=golang-dev, 0xe2.0x9a.0x9b, ality, rsc, rsc
CC=golang-dev
https://golang.org/cl/5700044

binary: remove unnecessary dummy type.

R=rsc
CC=golang-dev
https://golang.org/cl/5698052

godoc: don't show testdata directories

R=golang-dev, bradfitz, dsymonds, minux.ma, rsc
CC=golang-dev
https://golang.org/cl/5700043

go/parser: expand test cases for bad import

R=gri
CC=golang-dev
https://golang.org/cl/5697047

runtime: darwin signal masking

Fixes #3101 (darwin).

R=golang-dev, bradfitz
CC=golang-dev
https://golang.org/cl/5693044

runtime: linux signal masking

Fixes #3101 (Linux).

R=golang-dev, bradfitz, minux.ma
CC=golang-dev
https://golang.org/cl/5696043

cc: fix an out of bounds array access

Alternatively, we could expand the ewidth array
in [568]c/txt.c to have NALLTYPES elements and
give all types above NTYPE a width of -1.

I don't think it's worth it since TDOT and TOLD
are the only two type values above NTYPE that
are passed to typ:

$ /tmp/cctypes
cc/dcl.c:683:  t->down = typ(TOLD, T);
cc/dcl.c:919:  return typ(TDOT, T);
$

Fixes #3063.

R=golang-dev, rsc
CC=golang-dev
https://golang.org/cl/5694047

math: slightly more readable comments

Replaced /*-style comments with line comments;
there are two many *'s already in those lines.

R=golang-dev, rsc
CC=golang-dev
https://golang.org/cl/5699051

6a/6l: add IMUL3Q and SHLDL

Although Intel considers the three-argument form of IMUL to be a
variant of IMUL, I couldn't make 6l able to differentiate it without
huge changes, so I called it IMUL3.

R=rsc
CC=golang-dev
https://golang.org/cl/5686055

go spec: import path implementation restriction

R=rsc, r, r, adg
CC=golang-dev
https://golang.org/cl/5688074

test: commentary for [h-m]*.go

R=golang-dev, gri
CC=golang-dev
https://golang.org/cl/5674112

exp/types: fix package comment

R=golang-dev, dsymonds
CC=golang-dev
https://golang.org/cl/5695046

godoc: remove dead code

R=golang-dev, dsymonds
CC=golang-dev
https://golang.org/cl/5698044

go/parser: check import path restrictions

Replaces pending CL 5674097.
Thanks to ality@pbrane.org for spearheading
the effort.

R=rsc, r
CC=golang-dev
https://golang.org/cl/5683077

go cmd: print more than one error when running go test

Fixes #3055.

R=rsc, r
CC=golang-dev
https://golang.org/cl/5683079

tag weekly.2012-02-22

R=golang-dev
CC=golang-dev
https://golang.org/cl/5699044

weekly.2012-02-22

R=golang-dev, r, bradfitz
CC=golang-dev
https://golang.org/cl/5698043

net/http: some more cookie tests

Including a commented-out test we might
tackle later, after seeing what browsers do.

R=dsymonds, rsc
CC=golang-dev
https://golang.org/cl/5694045

encoding/xml: add example and docs for anon fields

Anonymous pointer fields is not yet supported.
The problem is documented in issue 3108.

R=golang-dev, rsc
CC=golang-dev
https://golang.org/cl/5694043

net: add a bit clarified multicast listener tests

Also adds -external flag to allow use of external networks on tests.

R=rsc
CC=golang-dev
https://golang.org/cl/5693043

pprof: add Profile type

Makes it possible for client code to maintain its own profiles,
and also reduces the API surface by giving us a type that
models built-in profiles.

R=golang-dev, r
CC=golang-dev
https://golang.org/cl/5684056

runtime: goroutine profile, stack dumps

R=golang-dev, r, r
CC=golang-dev
https://golang.org/cl/5687076

encoding/xml: move Unmarshal example to function

This also fixes MarshalIndent's example after the
recent formatting convention changes.

Fixes #2831.

R=golang-dev, rsc
CC=golang-dev
https://golang.org/cl/5671062

time: run full TestTicker test even during short tests

R=golang-dev, r, bradfitz, r
CC=golang-dev
https://golang.org/cl/5689076

5a: fix same arm bug

R=golang-dev, minux.ma
CC=golang-dev
https://golang.org/cl/5689073

5c, 5g, 5l: fix arm bug

Using reg as the flag word was unfortunate, since the
default value is not 0 but NREG (==16), which happens
to be the bit NOPTR now.  Clear it.

If I say this will fix the build, it won't.

R=golang-dev, r
CC=golang-dev
https://golang.org/cl/5690072

cmd/dist: fix install cmd/5g on non-arm system

R=golang-dev, r
CC=golang-dev
https://golang.org/cl/5689072

net: make raw IP tests robust

Make it rely on underlying socket's address family.

R=golang-dev, rsc
CC=golang-dev
https://golang.org/cl/5673091

net: make parseProcNetIGMP more robust

Suggested by Paul Borman.

Fixes #2826.

R=rsc, borman
CC=golang-dev
https://golang.org/cl/5689048

os: make the system info a value not a pointer on unix
fix a couple of other minor related details.

R=golang-dev, rsc
CC=golang-dev
https://golang.org/cl/5690071

builder: reuse existing workspace if possible

R=golang-dev, bradfitz, rsc
CC=golang-dev
https://golang.org/cl/5690069

net: delete debugging print

Fixes #3030.

R=golang-dev, bradfitz
CC=golang-dev
https://golang.org/cl/5689071

builder: update for os.Wait changes.

This compiles again.

R=golang-dev, minux.ma, rsc
CC=golang-dev
https://golang.org/cl/5687078

go/printer, gofmt: improved comma placement

Not a Go 1 issue, but appeared to be fairly easy to fix.

- Note that a few existing test cases look slightly worse but
  those cases were not representative for real code. All real
  code looks better now.

- Manual move of the comment in go/scanner/example_test.go
  before applying gofmt.

- gofmt -w $GOROOT/src $GOROOT/misc

Fixes #3062.

R=rsc
CC=golang-dev
https://golang.org/cl/5674093

godoc: proper index range checking when extracting example code

Fixes #3092.

R=golang-dev, rsc
CC=golang-dev
https://golang.org/cl/5683075

cgo: fix definition of opaque types

Fixes #3082.

R=golang-dev, rsc
CC=golang-dev
https://golang.org/cl/5683074

go/doc: move firstSentence into go/doc

- renamed firstSentence -> Synopsis
- also deal with common abbreviations

R=rsc
CC=golang-dev
https://golang.org/cl/5676088

net: replace error variable name oserr with err

This CL replaces the last two in source tree.

R=golang-dev, r
CC=golang-dev
https://golang.org/cl/5685080

godoc: fix potential index out-of-bounds error

R=golang-dev, bradfitz, dsymonds
CC=golang-dev
https://golang.org/cl/5683072

ld: get symbol table letters right

Have to assign them before we create the symbol table.
ARM debugging.

TBR=r
CC=golang-dev
https://golang.org/cl/5689067

gc: new, less strict bool rules

R=ken2
CC=golang-dev
https://golang.org/cl/5688064

gc: never crash during a debugging print

TBR=lvd
CC=golang-dev
https://golang.org/cl/5686063

spec: refine the wording about variables in type switches

R=golang-dev, rsc
CC=golang-dev
https://golang.org/cl/5686064

cmd/go: add tool -n flag

As in gdb $(go tool -n 6g).

R=golang-dev, r
CC=golang-dev
https://golang.org/cl/5689066

cmd/fix: remove os.Wait gofix.

The os.Wait function has been removed entirely, so there's no point in fixing code that called it.

R=r
CC=golang-dev
https://golang.org/cl/5685078

runtime: fix FreeBSD signal handling around thread creation
Ignore signals while we are spawning a new thread. Previously, a
signal arriving just before runtime.minit setting up the signal
handler triggers a "double fault" in signal trampolining.
Fixes #3017.

R=rsc, mikioh.mikioh, minux.ma, adg
CC=golang-dev
https://golang.org/cl/5684060

misc/dist/windows: fix broken icon, other misc

fix, and shrink, the gopher icon, txt extensions
for license and readme files, minor format changes
in the wxs file

R=golang-dev, rsc
CC=golang-dev
https://golang.org/cl/5683054

ld: only set SNOPTRBSS during load

dodata will convert to SNOPTRDATA if appropriate.
Should fix arm build (hope springs eternal).

TBR=golang-dev
CC=golang-dev
https://golang.org/cl/5687074

doc: move styles to style.css

R=golang-dev, rsc
CC=golang-dev
https://golang.org/cl/5685075

gc: return of ideal bool

This is a manual undo of CL 5674098.
It does not implement the even less strict spec
that we just agreed on, but it gets us back where
we were at the last weekly.

R=ken2
CC=golang-dev
https://golang.org/cl/5683069

ld: add NOPTRBSS for large, pointer-free uninitialized data
cc: add #pragma textflag to set it
runtime: mark mheap to go into noptr-bss.
        remove special case in garbage collector

Remove the ARM from.flag field created by CL 5687044.
The DUPOK flag was already in p->reg, so keep using that.

Otherwise test/nilptr.go creates a very large binary.
Should fix the arm build.
Diagnosed by minux.ma; replacement for CL 5690044.

R=golang-dev, minux.ma, r
CC=golang-dev
https://golang.org/cl/5686060

spec: make all comparison results untyped bool

Or, depending on your point of view, make the
comparisons satisfy any surrounding boolean type.

Also, fix a few foo_bar -> fooBar in code fragments.

Fixes #2561.

R=golang-dev, r, bradfitz, gri, iant, kevlar
CC=golang-dev
https://golang.org/cl/5671096

godoc: fix absolute->relative mapping

Fixes #3096.

R=golang-dev, dsymonds
CC=golang-dev
https://golang.org/cl/5690063

runtime: use SYSCALL instead of INT 0x80 in FreeBSD's runtime.osyield on amd64

R=mikioh.mikioh, rsc
CC=golang-dev
https://golang.org/cl/5690062

godoc: adjust line height in pre blocks

R=gri, dsymonds
CC=golang-dev
https://golang.org/cl/5687071

net/http: update test to use time.Duration better

R=golang-dev, dsymonds
CC=golang-dev
https://golang.org/cl/5688063

net/textproto: accept bad MIME headers as browsers do

Accept certain non-compliant response headers
(in particular, when spaces preceed the colon).
All major browser and curl seem to support this,
and at least one webserver seems to send these.

*shrug*

R=golang-dev, gri
CC=golang-dev
https://golang.org/cl/5690059

test: use testlib in a few more cases.

R=golang-dev, rsc
CC=golang-dev, remy
https://golang.org/cl/5688057

godoc: fix playground url

R=golang-dev, gri
CC=golang-dev
https://golang.org/cl/5685070

text/tabwriter: fix documentation by adding an example.

R=rsc, r
CC=golang-dev
https://golang.org/cl/5685069

misc/goplay: remove redundant chdir

R=ajstarks, r
CC=golang-dev
https://golang.org/cl/5687070

godoc: support flat directory view again

R=bradfitz
CC=golang-dev
https://golang.org/cl/5690058

doc: support redirect-on-share

R=golang-dev, rsc
CC=golang-dev
https://golang.org/cl/5689056

cmd/dist: fix pprof permissions

When installing pprof into the tools directory, it needs to
have execute permissions on unix-like systems.

Fixes issues 3077.

R=golang-dev, rsc, minux.ma
CC=golang-dev
https://golang.org/cl/5675095

net/smtp: use EHLO then HELO

Before we were using "ESMTP" in the banner as a clue,
but that is not required by the RFC and breaks mailing
to smtp.yandex.ru.

Fixes #3045.

R=golang-dev, bradfitz
CC=golang-dev
https://golang.org/cl/5687066

runtime: avoid malloc during malloc

A fault during malloc might lead to the program's
first call to findfunc, which would in turn call malloc.
Don't do that.

Fixes #1777.

R=golang-dev, gri
CC=golang-dev
https://golang.org/cl/5689047

codereview: add submit message support for newer clone URL and subrepos

R=golang-dev, rsc
CC=golang-dev
https://golang.org/cl/5689063

cmd/go: pass 2, remove reference to incomplete -file flag

R=golang-dev, minux.ma, rsc
CC=golang-dev
https://golang.org/cl/5686058

cmd/go: 'go help test' remove documentation for incomplete -file flag

R=golang-dev, rsc, r, r
CC=golang-dev
https://golang.org/cl/5673093

cmd/*: add -d option to bison.

Without -d, bison doesn't generate y.tab.h.

R=rsc
CC=golang-dev
https://golang.org/cl/5685065

exp/norm: Added Iter type for iterating on segment boundaries.  This type is mainly to be used
by other low-level libraries, like collate.  Extra care has been given to optimize the performance
of normalizing to NFD, as this is what will be used by the collator.  The overhead of checking
whether a string is normalized vs simply decomposing a string is neglible.  Assuming that most
strings are in the FCD form, this iterator can be used to decompose strings and normalize with
minimal overhead.

R=r
CC=golang-dev
https://golang.org/cl/5676057

test: fix bug423.go to actually fail with older releases.

The supposedly overflowing variable was registerized.

R=golang-dev, r
CC=golang-dev
https://golang.org/cl/5687061

cmd/api: record return type of functions for variable typecheck.

Also cleanup the resolveName method.

Fixes failure on go/build declaration:
        var ToolDir = filepath.Join(...)

R=golang-dev, bradfitz
CC=golang-dev, remy
https://golang.org/cl/5681043

dashboard: rename buttons to satisfy pedantic souls

R=golang-dev, r
CC=golang-dev
https://golang.org/cl/5686053

net: extend the workaround to DialTCP to try to get arm building again.
Awfulness by the bucket that we hope
Fixes #3057.

R=golang-dev, mikioh.mikioh, dsymonds, r, rsc
CC=golang-dev
https://golang.org/cl/5687060