release.2011-02-15

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

encoding/line: fix line returned after EOF

Fixes #1509.

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

godoc: don't hide package lookup error if there's no command with the same name

Fixes #1514.

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

rpc: properly discard values.

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

runtime: detect failed thread creation on Windows

Fixes #1495.

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

makehtml: use append
The program is old and missed its opportunity.

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

tutorial: rework the introduction to give "Effective Go"
prominence and downplay the course notes.

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

archive/zip: handle files with data descriptors

Fixes #1471.

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

gob: decode into nil, this time for sure.

Yesterday's change was too simple-minded and failed if an
interface value was being discarded.  We need to parse the
data stream and remember any type information that arrives.

Also fix a minor bug when ignoring an interface: toss only what
we know about, not everything.

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

windows: runtime: implemented console ctrl handler (SIGINT).

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

build: run test/ directory first

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

go/printer: line comments must always end in a newline

Fixes #1503.

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

gob: allow Decode(nil) and have it just discard the next value.
Fixes #1489.

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

code.html: update to reflect that package names need not be unique

Fixes #1507.

R=golang-dev, ehog.hedge, rsc
CC=golang-dev
https://golang.org/cl/4160049

5ld: part of 64bit eor - forgot to check in.

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

gofmt: exclude testcase with incorrect syntax

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

strconv/ftoa: avoid a double shift. (shifts by variables are expensive.)

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

testing: include elapsed time in output

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

gc: correct receiver in method missing error

Fixes #1324.

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

sync: check Unlock of unlocked Mutex

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

5ld: stoped generating 64-bit eor

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

syscall: remove obsolete socket IO control

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

http: add pipelining to ClientConn, ServerConn

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

syscall: add sockaddr_dl, sysctl with routing message support for darwin, freebsd

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

runtime: check that SysReserve returns aligned memory

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

8l, runtime: place G and M pointers relative to _tos on Plan 9, instead of hardcoded values for USTKTOP.

This should allow executing both on native Plan 9 and inside 9vx.

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

bytes: fix bugs in buffer.ReadBytes

Fixes #1498.

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

crypto/openpgp: minor updates to subpackages

Now that packet/ is checked in, we can add its Makefile. Also, a couple
of updates to error/ and s2k/ for bugfixes and to use the new crypto
package.

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

fix example in inotify

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

flag: allow hexadecimal and octal input for integer flags.

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

crypto/openpgp/packet: add remainder of packet types.

(The unittest for Signature may seem a little small, but it's tested by
the higher level code.)

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

godoc: godoc: Use IsAbs to test for absolute paths (fix for win32).

One more case.

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

godoc: Use IsAbs to test for absolute paths (fix for win32).

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

syscall: change windows apis with bool return value to return errno instead

This change is to make these apis similar to their unix counterparts.

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

http: add Server type supporting timeouts

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

runtime: complete windows SysReserve

Should fix windows/386 build.

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

encoding/binary: update package doc string to be more comprehensive

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

fmt: fix minor typo

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

json: correct Marshal documentation

Fixes #1488.

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

debug/pe: ImportedSymbols fixes

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

crypto/openpgp/packet: four more packet types.

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

runtime: take the callback return value from the stack

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

src/*: fix incorrect prints found by govet

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

peep: more bugs

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

buffer.go: minor optimization, expanded comment

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

godoc: optimizations: don't call Write for 0-length data

- guard some calls to Write that frequently may have 0-length data
- fix an invariant

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

template: reverse order of arguments to Execute

In line with other functions such as Fprintf, put the
thing to be written first.

Apologies for the breakages this is sure to cause.

R=rsc, gri, adg, eds, r2, aam
CC=golang-dev
https://golang.org/cl/4169042

srcextract: HTML-escape output if so desired

This functionality was removed with CL 4169041.

Minor simplifications.

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

peep: bug fix

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

run.bash: must make codelab before we can test it.
this will break the build, but it's already silently broken.

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

codelab: update due to recent changes in go/printer

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

runtime: fix memory allocation on 386

BSD and Darwin require an extra page between
end and the first mapping, and Windows has various
memory in the way too.

Fixes #1464.

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

runtime: new allocation strategy for amd64

Do not reserve virtual address space.
Instead, assume it will be there when we need it,
and crash loudly if that assumption is violated.
Reserving the address space gets charged to
ulimit -v, which exceeds commonly set limits.

http://groups.google.com/group/golang-dev/msg/7c477af5f5a8dd2c

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

syscall: do not use NULL for zero-length read, write

Avoids problems running Linux binaries under QEMU.

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

rpc: make more tolerant of errors.
Add Error type to enable clients to distinguish
between local and remote errors.
Also return "connection shut down error" after
the first error return rather than returning the
same error each time.

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

go/printer: remove notion of "Styler", remove HTML mode

Neither gofmt nor godoc are making use of a Styler (for
token-specific formatting) anymore. Stylers interacted in complicated
ways with HTML-escaping which was why the printer needed an HTML mode
in the first place.

godoc now uses a more powerful and general text formatting
function that does HTML escaping, text selection, and can
handle token-specific formatting if so desired (currently
used only for comments).

As a consequence, cleaned up uses of go/printer in godoc;
simplified the various write utility functions, and also
removed the need for the "html" template format (in favor of
html-esc which now does the same and is used more pervasively).

Applied gofmt -w src misc to verify no changes occured,
and tested godoc manually.

There should be no visible changes except that (type) code
snippets presented for godoc package documentation now
uses the same formatting as for general source code and
thus comments get the comment-specific color here as well
(not the case at the moment).

(TODO: godoc needs a good automatic test suite).

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

crypto/x509: add name constraints support.

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

misc/dashboard: hide benchmarks link temporarily

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

build: Drop syslog on DISABLE_NET_TESTS=1

Even if local, it requires communication with a daemon
which may not be available.  This is creating problems
for getting an Ubuntu package going in Launchpad's PPA.

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

cgo: fix handling of signed enumerations

Structs defined in C as containing a field with
an enum type are currently translated to Go as
a struct with an unsigned integer field, even if
some of the values contained in the enum are
negative.

This modification takes in consideration the values
defined in the enum, and conditionally defines the
Go type as signed if necessary.

The logic introduced was tested with gcc, which
will increase the type size if it contains both
negative numbers and values greater than 2^b/2-1,
and refuses to compile values which would be
problematic (2^64-1, but in fact the ISO C
restricts the range to the size of int).

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

http: handle unchunked, un-lengthed HTTP/1.1 responses

Fixes #716

This CL simply resumes the previous CL in-flight at
https://golang.org/cl/906042/

R=rsc, petar-m, dsymonds
CC=golang-dev
https://golang.org/cl/4157042

container/ring: Replace Iter() with Do().

Faster in most cases, and not prone to memory leaks. Named "Do" to match with similarly named method on Vector.

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

syscall: implement windows version of Fsync

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

fix build clean.bash

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

codereview: use cmd.communicate.

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

src/run.bash: get rid of long windows expression

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

.hgignore: ignore src/pkg/runtime/version_*.go

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

cgen64: dont use MVN instruction

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

peep: more optimization

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

go spec: fix a few typos

The spec can now be parsed with an
xml.Parser using the HTML settings.

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

go spec: clarification re: method sets of newly declared pointer types

- added an example to Type declarations section clarifying the
  situation brought up with issue 1324
- slightly re-ordered paragraphs in Types section
- added separate heading for method set section and refer to it
  from elsewhere in the spec
- no language changes

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

windows: replace remaining __MINGW32__ instances with _WIN32

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

netchan: graceful handling of closed connection
Currently, when an importer closes the connection, the exporter gives an
error message 'netchan export: error decoding client header:EOF'.  This
change causes the exporter to look for an EOF during the parse of the
header, and silences the log message in that case.

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

cgo: don't run cgo when not compiling

The logic introduced to avoid running cgo when
introducing _cgo_flags is faulty.  My goal was
to handle it with the following statement:

    -include _cgo_flags

The dash tells make to ignore errors if it can't
include the file.

What I missed, though, was the fact that it
*will* attempt to build the file if it knows
how.

This change will introduce the originally
intended semantics of not attempting to build
the file before necessary.

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

crypto/openpgp/packet: add UserId packet type

This was split from CL 4124054

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

peep: fix optimization bug

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

peep: fix bug in peep optimizer.
reg: enable peep optimizer.
cgen64: better int64 code.

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

syscall: fix arm build

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

ld: fix build (uvlong vs uint64 conflict on freebsd)

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

loader: move the XputY routines into the ld directory.
Fixes the build for 5l, and also removes an inconsequential bug in 8l.

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

Codelab/wiki: fix typo

Missing closing bracket renders the next code snippet unreadable.

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

Codelab: correct function definitions for handlers before closures are introduced.

A couple of post-closure function definitions were introduced too early, making the resulting
code fail compilation.

Also, the TitleValidator regexp was missing.

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

crypto/tls: select best ciphersuite, not worst.

Previously, the outer loop would continue until we selected the
client's least preferable ciphersuite.

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

asn1: marshal true as 255, not 1.

OS X, at least, appears to test |byte == 255|, not |byte != 0| to
establish if a bool is true or false.

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

crypto/tls: load a chain of certificates from a file.

Many recently issued certificates are chained: there's one or more
intermediate certificates between the host certificate and the root CA
certificate. This change causes the code to load any number of
certificates from the certificate file. This matches the behaviour of
common webservers, and the output of OpenSSL's command line tools.

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

crypto/openpgp/packet: two more packet types.

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

go/printer, gofmt: smarter handling of multi-line raw strings

If a multi-line raw string is the first token on a line, it
should not be indented because the following lines (belonging
to the raw string) are not indented either.

Adjusted src of ebnf/ebnf_test.go manually as it now is formatted
as expected.

gofmt -w src misc

Fixes #1072.

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

template: Add simple formatter chaining.

Fixes #676.

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

gofmt: no need for lexical compare of src and res (optimization)

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

template: allow a leading '*' to indicate that evaulation should
indirect through a pointer.

Fixes #1478.

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

scan: permit base prefixes 0nnn and 0xnn when scanning
signed or unsigned integers using %v or the formatless scanner.
That is, Sscan("0x11", &i) or Sscanf("0x11", "%v", &i) will now
set i to 17.   If a format other than %v is presented, the behavior
is as before.

Fixes #1469.

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

8l: fix crash writing Plan 9 binaries

Was crashing with GOOS=plan9 unless -s was passed.
Add symbols and line numbers to Plan 9 a.out.

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

build: allow clean.bash to work on fresh checkout

Must be invoked as ./clean.bash --gomake make
(or --gomake gmake, depending on the name of
GNU make).

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

runtime: fix asm.h on Windows

Thanks to mhantsch@gmail.com.

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

spec: fix Typeof() return type

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

go spec, effective go: cleanups

Removed most of the detailed examples about handing panics
from the go spec since it's now covered by Effective Go.

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

crypto/openpgp/packet: add public key support

Note that DSA public key support is nascent and the verification
functions clearly don't support it yet. I'm intending to get RSA keys
working first.

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

issue 1402: added test case

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

os: implement new Process api

Fixes #1004.
Fixes #1460.

R=mattn, r, niemeyer, rog, rsc
CC=golang-dev
https://golang.org/cl/4029053

gopack: unused variable.

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