release.2010-09-22

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

netchan: fix comment typo.

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

exp/draw: add Point.Eq, Point.Mul, Point.Div, Rectangle.Size methods

R=nigeltao, nigeltao_golang, rsc
CC=golang-dev, r
https://golang.org/cl/2192048

http: fix redirect test for international users

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

build: add missing files for arm changes

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

http: Change redirect test URL, as the old one now fails.

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

gofmt: support for ... after actual arguments

Pending acceptance of the proposed language change.

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

suffixarray: cleanup per suggestion from Roger Peppe

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

http: don't always escape all reserved chars (fix build)

R=nigeltao, nigeltao_golang
CC=golang-dev
https://golang.org/cl/2206044

http: escape _all_ reserved characters as per the RFC

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

suffixarray: a package for creating suffixarray-based indexes

This is a replacement for pending CL 2219042. It only contains
the raw suffixarray functionality with two methods:

- New       create a new index from some data
- Lookup    lookup occurences of a bytes slice in the data

Any other functionality (dealing with multiple data sets and
the corresponding position lists) is generic and doesn't have
to be part of this package.

Known performance bug: This implementation works fine for data sets
up to several megabytes as long as it doesn't contain very long
contiguous sequences of equal bytes. For instance, index creation for
all .go files under GOROOT (250KLOCs, approx. 9MB) takes ~50s on
2.66 GHz Intel Xeon as long as test/fixedbugs/257.go is excluded.
With that file, index creation times takes several days. 257.go contains
a string of 1M smiley faces.

There are more sophisticated suffixarray creation algorithms which
can handle very long common prefixes. The implementation can be
updated w/o the need to change the interface.

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

fmt: support '*' for width or precision

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

build: make all.bash run on Ubuntu ARM

Sent from my phone.

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

misc/dashboard/builder: -cmd for user-specified build command

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

os: make Open() O_APPEND flag work on windows

Fixes #1124.
Implementation is suggested by Skip.
Test is suggested by PeterGo.

R=r, PeterGo, rsc
CC=golang-dev, skip.tavakkolian
https://golang.org/cl/2256041

strings: fix minor bug in LastIndexFunc

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

build: fixes for native arm build

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

cgo: bug fixes

* Add documentation about array arguments.  Fixes issue 1125.
* Do not interpret x, y := z, w as special errno form.  Fixes issue 952.
* Fix nested Go calls (brainman).  Fixes issue 907.

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

exp/4s, exp/nacl/av: sync to recent exp/draw changes.

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

misc/dashboard/builder: fix bug that caused old revisions to be rebuilt

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

doc: variable-width layout

The front page remains fixed-width.
All other pages should look good in windows >=500px wide.

Includes additional styles for search result highlighting,
code comments, and general CSS clean-ups.

Tested with Chrome 6, Firefox 3.6, IE 7, and IE 8.

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

syscall: Use vsyscall for syscall.Gettimeofday and .Time on linux amd64.

Regenerate zsyscall_linux_*.go files with recent changes to
mksyscall.sh.

Add socketpair to syscall_linux_amd64.go; for some reason it
was in the generated file but not in the source file.

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

regexp: delete Iter methods
They are unused and not that useful anyway.

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

misc/dashboard/builder: fixes and improvements

- fix release upload
- add -rev= flag to build specific revision and exit
- added support for all-$GOARCH.bash

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

env.bash: fix building in directory with spaces in the path.

Fixes #1122.
Thanks to BigBoss1964 for the solution.

R=golang-dev, r2
CC=golang-dev
https://golang.org/cl/2246042

exp/nacl/av: update color to max out at 1<<16-1 instead of 1<<32-1.
Fix missing return in Set(x, y, color).

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

godoc: default filter file name should be ""

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

loader fix
static init redo

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

doc: fix typo in go_spec.html

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

6l/8l: emit DWARF frame info.

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

crypto/x509, crypto/tls: improve root matching and observe CA flag.

The key/value format of X.500 names means that it's possible to encode
a name with multiple values for, say, organisation. RFC5280
doesn't seem to consider this, but there are Verisign root
certificates which do this and, in order to find the correct
root certificate in some cases, we need to handle it.

Also, CA certificates should set the CA flag and we now check
this. After looking at the other X.509 extensions it appears
that they are universally ignored/bit rotted away so we ignore
them.

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

Fix certificate validation.

asn1: add support for T61String because this is the string type which
    several www.google.com certificates are now using for fields like
    CommonName
tls: force a handshake in Dial so that certificates are ready
    afterwards.

Fixes #1114.

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

netchan: fix unimportant typo in test error call.

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

netchan: provide a method (Importer.Errors()) to recover protocol errors.

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

Make.inc: make GOOS detection work on windows

R=rsc, Joe Poirier, r
CC=golang-dev
https://golang.org/cl/2248041

syscall: fix NaCl

missing from last CL, sorry

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

test: fix NaCl build

Wasn't running the temp binary under $E (emulator).

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

runtime: better panic for send to nil channel

*Much* better on NaCl, where memory faults are deadly.

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

nacl: fix zero-length writes

NaCl rejects zero-length write using nil pointer.

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

6l, 8l: clean up ELF code, fix NaCl

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

exp/draw/x11: mouse location is a signed integer.

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

netchan: handle closing of channels.
This also silences some misleading logging.
Also improve logging.

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

5l: avoid fixed buffers in list

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

doc/go_mem.html: update location of "once".
Fixes #1118.

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

gob: make robust when decoding a struct with non-struct data.
The decoder was crashing when handling an rpc that expected
a struct but was delivered something else.  This diagnoses the
problem.  The other direction (expecting non-struct but getting
one) was already handled.

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

windows: fix netchan test by using 127.0.0.1.
Re-enable the test on Windows.

R=brainman, alex.brainman
CC=golang-dev
https://golang.org/cl/2244041

pkg/Makefile: disable netchan test to fix windows build

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

doc: fix typo in go_for_cpp_programmers.html

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

os: make RemoveAll() work on windows

R=golang-dev, rsc1
CC=golang-dev
https://golang.org/cl/2212041

fix various comments

(os package comments were duplicates)

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

add readonly flag to static initialization

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

netchan: add a cross-connect test.
Suggested in a bug report (since addressed) by Axel Schreiner.

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

godoc: show "Last update" info for directory listings.

Use -timestamps=false flag to disable.

(This used to be shown on the front-page below the
build information with the old godoc. However, the
time stamps are directory-specific and should be
shown with the directory.)

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

gc: spell debug['u'] correctly

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

gc: make sure path names are canonical

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

gofmt: add another exception to test cases

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

godoc: don't use quadratic algorithm to filter paths

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

cgo: show preamble gcc errors

Fixes #1081

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

net/dict: parse response correctly + typo

* SHOW DB responds with a
  "110 n databases present - text follows" -- parse it.
* Doing a Define() on a non-existing word gives error
  "invalid definition count: no", when we really
  want "552 no match".

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

tag release.2010-09-15

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

release.2010-09-15

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

golang.org redesign

R=r, rsc, gri
CC=golang-dev, iant, ken2, lvd, nigeltao_golang, stephenm
https://golang.org/cl/2182042

netchan: fix bug for imported send.
Also add a bit of debugging and sanitation code.
Fixes #769.

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

fix build

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

os: error in documentation

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

godoc: only show directories containing true package files

(ignore directories containing *.go files that don't
actually start with a package clause)

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

fix line number printing
with //line directives.

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

libmach: fix build on arm host

Fixes #1079.

R=r, trufae, fgergo
CC=golang-dev
https://golang.org/cl/2103048

netchan: make -1 unlimited, as advertised.

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

godoc: better handling of deep directory trees

also: fix a logic error with filter use at startup

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

godoc: atomically update filter file

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

godoc: use correct delay time (bug fix)

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

godoc documentation: fixed typo, more precise comment

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

godoc: better support for directory trees for user-defined
       file systems provided via -path

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

tabs

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

6l, 8l: make etext accurate; introduce rodata, erodata.

Makes binaries work with 6cov again.

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

Preliminary support for 'copy' builtin function in exp/eval

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

CLA for Sebastien Binet

R=adg
CC=binet, golang-dev
https://golang.org/cl/2188041

doc: fix typo in Programming FAQ

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

fmt.Fscan: use UnreadRune to preserve data across calls.
Fixes #1055.
(About as well as it can be fixed without changing the API.)

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

gc: implement new comparison rule

The new comparison rule was added to the spec by

changeset:   5605:33abb649cb63
user:        Robert Griesemer <gri@golang.org>
date:        Thu Jun 03 16:55:50 2010 -0700
files:       doc/go_spec.html
description:
go spec: Base comparison compatibility on assignment compatibility.

Specifically:
- Simplified definition of comparison compatibility and folded into
  section on comparison operators since it's only used there.

This is a small language change/cleanup. As a consequence:
- An interface value may now be compared against a non-interface value.
- Channels with opposite directions cannot be compared directly anymore
  (per discussion with rsc).

R=rsc, r, iant, ken2
CC=golang-dev
https://golang.org/cl/1462041

but never implemented.

Fixes #1070.

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

gc: bug306

Fixes #1040.

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

A+C: Kai Backman (individual CLA)

Just AUTHORS; already in CONTRIBUTORS from work at Google.

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

json: Add package comment

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

go-mode.el: Fix highlighting for 'chan' type
Fixes #1038.

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

(windows) disable tests that cause the build to fail

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

arm tests: disable zerodivide.go: compilation fails.
Hope to fix the build.

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

libcgo: fix NaCl build.

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

goinstall: warn when http:// is found at beginning of package name
Fixes #860.

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

misc/dashboard/builder: gobuilder, a continuous build client

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

gc: fix symbol table generation on windows

gc records full, '/' delimited, filenames now.

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

bufio: add UnreadRune.

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

exec.LookPath() unix/windows separation

R=brainman, rsc, vcc, rsc1
CC=golang-dev
https://golang.org/cl/2068041

libmach: fix whitespace.

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

test: Make gccgo believe that the variables can change.

This test is supposed to test runtime behaviour.  However,
gccgo is smart enough to see that the variables are local to
the package and that they never change.  Add a function so
that gccgo thinks that the variables can in fact vary, so that
the test works at runtime as expected.

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

gc: make string x + y + z + ... + w efficient

1 malloc per concatenation.

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

6g, 6l, 8g, 8l: move read-only data to text segment

Changing 5g and 5l too, but it doesn't work yet.

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

5g: register allocation bugs

Fixes #1099.

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

codereview: convert email address from Rietveld to lower case
before looking for it in the CONTRIBUTORS file

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

libmach: fix new thread race with Linux

If you look at the sequence of values returned
by waitpid, it simply tells us about the child of
clone before it tells us that the parent called clone.
There's nothing we can do but assume unexpected
tids are newly cloned children.

Tested with 6prof on godoc.

Fixes #251.

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