release.2010-08-04

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

path: fix comment for Split()

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

go/parser.ParseFiles: don't override error (to nil) if there is one

Also: Return first instead of last error as that seems more useful.

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

fix build: literal types cannot be parenthesized anymore

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

amd64: use segment memory for thread-local storage
Returns R14 and R15 to the available register pool.
Plays more nicely with ELF ABI C code.
In particular, our signal handlers will no longer crash
when a signal arrives during execution of a cgo C call.

Fixes #720.

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

misc/vim: highlight misspelled words only in comments

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

gofmt/go/parser: strengthen syntax checks

- don't allow parenthesized receiver base types or anonymous fields
- fixed a couple of other omissions
- adjusted gofmt test script
- removed several TODOs

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

goinstall: check for error from exec.*Cmd.Wait() before using its result

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

cgo: generate correct multiple return value function invocations

Fixes #967.

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

A+C: fix email address

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

gofmt: make code more readable by avoiding redeclaration

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

bytes: added test case for explode with blank string.
Tests for the equivalent of the strings.explode("") panic bug (issue 980).

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

json: object members must have a value

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

A+C: Anthony Martin, Christian Himpel (individual CLA)

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

bufio: introduce Peek.

I'll leave whether or not this obsoletes UnreadByte for a future CL.

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

os: change the type of permissions argument for Open etc. to uint32.
Besides being more correct, it protects against people accidentally
exchanging the permission and open mode arguments to Open.

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

websocket: correct challenge response

Tested against latest Chrome.

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

bikeshed: only output bug when we have a regression

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

CONTRIBUTORS: Rietveld alias for brad@danga.com

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

os: Null device

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

arm: remove old qemu dependency from gotest

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

test/run: diff old new

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

crypto/x509: unwrap Subject Key Identifier

RFC 5280, 4.2.1.2 says:
  SubjectKeyIdentifier ::= KeyIdentifier
  KeyIdentifier ::= OCTET STRING

Previously, we were failing to unwrap the second level of OCTET STRING
encoding.

Fixes #993.

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

gc: empty select

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

gc: bug301

Fixes #990.

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

gc: bug299, bug300

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

A+C: Tarmigan Casebolt

R=adg
CC=golang-dev, tarm
https://golang.org/cl/1923041

gc: index bounds tests and fixes
move constant index checking to front end
x[2:1] is a compile-time error now too

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

strings: fix Split("", "", -1)
Fixes #980.

Made it return an empty array, rather than crash.
Added relevant test cases to strings.

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

Add Scott Lawrence to C&A

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

syscall: make sure go error numbers do not clash with windows system errors

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

builder: update script so it doesn't create tar-bombs

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

image: introduce Gray and Gray16 types, and remove the named colors
except for Black and White.

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

io: consolidate multi_reader and multi_writer into a single file, multi.go

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

image/png: use image-specific methods for checking opacity.

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

websocket: fix bug involving spaces in header keys

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

CONTRIBUTORS: add wcn@google.com alias to match with code review

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

runtime: do not fall through in SIGBUS/SIGSEGV

Faults beyond the first page are not expected
and should fail loudly.  They are not subject to recover.

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

arm: disable another flaky test.

these tests work fine on n1 and gumstix. it's unclear
why they keep failing on the godashboard build.

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

fix build

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

arm: bugfixes (stack clobbering, indices)

also changed zerodivide to output "BUG"

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

arm: add gdb support to android launcher script

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

tag release.2010-07-29

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

release.2010-07-29

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

misc/builder: support for uploading tarballs to googlecode

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

cov and prof: implement windows version (just function stubs and build mods)

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

bug300: literal types must not be parenthesized

( This CL is dependent on acceptance of
https://golang.org/cl/1913041/show )

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

go_spec: don't allow parens around the literal type of composite literals

Background: The current spec is imprecise with respect to the parsing ambiguity
for composite literals: It says that the ambiguity arises when the TypeName form
of the LiteralType is used. The following code:

    if (B) {} ...

is not using the TypeName form (but the parenthesized TypeName form) and thus
could be interpreted as:

    if ((B){}) ...

instead of

    if B {} ...

Both compilers and gofmt choose the latter interpretation. One could fix the
spec by making the clause regarding the parsing ambiguity more precise ("...using
the _possibly parenthesized_ TypeName form of the LiteralType..."). The alternative
(chosen here) is to simply disallow parenthesized literal types. Except for a single
test case (test/parentype.go) there appears to be no Go code under $GOROOT containing
parenthesized literal types. Furthermore, parentheses are never needed around a
literal type for correct parsing.

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

doc: update front page and go_learning.html

This is a stop-gap change to give more current information visibility
before a more thorough reorganization.

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

mime: add AddExtensionType

For example:
mime.AddExtensionType(".m3u8", "application/x-mpegURL")
mime.AddExtensionType(".ts", "video/MP2T")

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

A+C: Yuusei Kuwana (individual CLA)

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

scanner: better comment

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

bug299: parenthesized receiver types/anonymous fields are illegal

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

fmt.Print: fix bug in placement of spaces introduced when ...T went in.
Fixes #976.

R=rsc, chris tighe, r
CC=golang-dev
https://golang.org/cl/1697057

runtime: fix bug introduced in revision 4a01b8d28570

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

net: implement windows version of LookupHost/Port/SRV

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

test/sigchld.go: use syscall.Kill instead of a combination
of syscall.Syscall and syscall.SYS_KILL.

In RTEMS, there is no syscall.Syscall support, but it does
support POSIX signals. So, if this testcase is changed to use
syscall.Kill, then it would run fine on RTEMS, when using gccgo.

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

gc: fix SIGBUS

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

http: add https client support

Fixes #851.

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

fmt: move comment into doc.go.
Fix the documentation of newline handling in *f routines
to match the implementation.

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

io: MultiReader and MultiWriter

Little helpers I've found useful.

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

arm: fix build (dodgy floats)

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

arm: minor bugfixes.

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

5g: fix build

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

bufio: fix optimisation test

Fixes #949.

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

net: add nil checks to several String methods to avoid panics

Fixes #945.

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

bytes, strings: mention the n < 0 case in Split/SplitAfter doc comment

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

spec: add recover to predeclared identifiers list

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

dashboard: more aggressive caching for project/package pages

Plus automatic package<->project association script.
(This is just a helper script for now. I intend to fully automate
the associations further down the track.)

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

gc: include signal.h (fix build)

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

godoc: accept '.', '!', and '?' as end of first sentence of package documentation

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

gc: more crash avoidance

Fixes #961.
Fixes #962.

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

6l: more invalid input files

Fixes #963.
Fixes #964.

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

codereview: avoid crash if no config

Fixes #959.

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

gc: graceful exit on seg fault

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

fmt.Scanf: handle trailing spaces.
Fixes #954.

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

runtime: fix goc2c for rename to goc2c and *.goc

Release 2010-04-27. runtime: rename cgo2c, *.cgo to goc2c, *.goc
to avoid confusion with real cgo.

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

net: TCPConn.SetNoDelay, back by popular demand

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

godoc: display synopses for all packages that have some kind of documentation.

Fixes #953.

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

gc: fix smaller-than-pointer-sized receivers in interfaces

Fixes #812.

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

gc: import dot shadowing bug

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

syscall: add windows version of Pipe()

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

prof: fix typo in usage string

Fixes #948.

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

A+C: Wei Guangjing

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

net: fix crashing Read/Write when passed empty slice on windows

Fixes #921.

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

syscall: improve windows errno handling

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

fix windows Make.cmd:
- TARG had extra space at the end;
- it should be set before "all" target.

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

syscall: add ForkExec, Syscall12 on Windows

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

runtime/extern.go: clear a small error in a comment.

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

bytes: port IndexFunc and LastIndexFunc from strings package

This CL basically applies the same changes as

http://code.google.com/p/go/source/detail?r=5e0a29014e8e

but for bytes package.

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

misc/vim: updated syntax file to recognize constants of the form 1e9

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

libmach: correct handling of .5 files with D_REGREG addresses
undo workaround in gc

Fixes #943.

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

fix build

bug introduced in https://golang.org/cl/1886043

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

test: fail is for bad exit status only

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

build: clean.bash to check that GOOS and GOARCH are set

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

gotest: fix the glob pattern in the top-of-file comment

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

ld: various bug fixes

Fixes #937.
Fixes #938.
Fixes #939.
Fixes #940.

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

strings.Bytes: fix typo in comment

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

io: Avoid another race condition in pipes.

Goroutine 1:
  Call Read on read half of pipe, entering pipeHalf.rw.
  Check ioclosed field, which is false.
  Send data to p.c1
  Wait for response on p.c2.

Goroutine 2:
  Call Close on read half of pipe, entering pipeHalf.close.
  Set closed field.
  Send error to p.cclose.
  Set ioclosed field.
  Send 1 to p.done.
  Return and exit goroutine.

Goroutine 3:
  This is the goroutine running pipe.run, and for some reason
  it has started late.
  Read error from p.rclose; set rerr and continue.
  Read 1 from p.done; increment ndone and continue.
  Read data from r1 (sent by goroutine 1); set r1 = nil and continue

Now goroutine 1 is waiting for a response, and goroutine 3 is
waiting for something else to happen.

This patch fixes the race by having the runner check whether
the read half is closed when it is asked for read data, and
similarly for the corresponding race on the write half.

This patch also fixes the similar race in which ndone gets
bumped up to 2 while there is a reader or writer waiting.

There is still another race to fix.  It is possible for the
read half and the write half to both be closed, and for the
runner goroutine to exit, all before the runner goroutine sees
the request from a reader.  E.g., in the above, have goroutine
2 also close the write half, and have goroutine 3 see both
done messages before it sees the request from goroutine 1.

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

syscall: On Windows, Errstr FormatMessage has no values to insert.

For the Windows version of syscall Errstr, set the
FORMAT_MESSAGE_IGNORE_INSERTS value of the FormatMessage
Flags argument when there are no values to insert.

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