release.2011-01-19

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

net: remove duplicate import of runtime

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

syscall: attempt to fix windows build, supply missing constant

R=r, brainman, rsc1
CC=golang-dev
https://golang.org/cl/4023043

crypto/twofish: update with rsc's comments

        TBR=rsc

R=rsc
CC=berengar.lehr, golang-dev
https://golang.org/cl/4050042

arm bug with stack adjust

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

codereview: fix hg change on Windows

hg change fails on Windows with 'No valid patches found in output from hg diff'.

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

godoc: enable fulltext index by default

- added flag -maxresults (default: 10000) to limit the max.
  number of full text results shown
- removed flag -fulltext; use -maxresults=0 to disable fulltext
  index
- better indication on result page if not all results are shown
  (... after line list)

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

godoc: enable qualified identifiers ("math.Sin") as query strings again

A query string of the form ident.ident will be used both as a qualified
identifier for identifier search and as a regular expression.
Qualified identifier lookup got broken accidentally when introducing
regexp full text search. Cleaned up surrounding logic a bit.

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

xml: handle tag paths through the same element

With the current implementation, xml unmarshalling
will silently fail to unmarshal any paths passing
through the same element, such as:

type T struct {
A string "dummy>a"
B string "dummy>b"
}

This change tweaks the algorithm so that this works
correctly.

Also, using paths that would cause the same element to
unmarshal twice will error out ahead of time explaining
the problem, rather than silently misbehaving.

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

runtime/debug: fix build (missing Makefile)
Why does this happen so often?

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

5a, 5l, 6a, 6l, 8a, 8l: handle out of memory, large allocations

Fixes #392.

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

A+C: Jeff R. Allen (individual CLA)

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

netchan: do not block sends; implement flow control.
When data is received for a channel, but that channel
is not ready to receive it, the central run() loop
is currently blocked, but this can lead to deadlock
and interference between independent channels.
This CL adds an explicit buffer size to netchan
channels (an API change) - the sender will not
send values until the buffer is non empty.

The protocol changes to send ids rather than channel names
because acks can still be sent after a channel is hung up,
we we need an identifier that can be ignored.

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

runtime/debug: new package
Facilities for printing stack traces from within a running goroutine.

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

net: fix build on freebsd

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

http: support for relative URLs

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

net: add LookupAddr

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

CONTRIBUTORS: secondary email for Kyle Lemons

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

windows: implement exception handling

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

8l: pe changes

Dll names and function names padded to even size.
Refactoring of imports writing code.

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

net: implement windows timeout

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

codereview: don't mail change lists with no files

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

net, syscall: return source address in Recvmsg

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

cgo: add complex float, complex double

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

math: handle denormals in Frexp, Ilogb, Ldexp, and Logb

Also:
* document special cases for Frexp and Ldexp
* handle ±Inf in Ldexp
* correctly return -0 on underflow in Ldexp
* test special cases for Ldexp
* test boundary cases for Frexp, Ilogb, Ldexp, and Logb

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

net: add unixpacket

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

regexp: reject bare ?

Minor cleanup:
  - removed a duplicate test case
  - added a function to remove repeated code
  - for consistency, replaced "return nil" with a panic at an
    unreachable point

Fixes #1428.

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

CONTRIBUTORS: add Ben Lynn (Google employee)

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

runtime: add per-pause gc stats

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

fmt: document %X of string, []byte

suggested by Mark Summerfield

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

go spec: adjust language for constant typing

Fixes #1421.

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

A+C: Clement Skau (individual CLA)

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

http: fix Content-Range and Content-Length in response

RFC2616 sections 4.4 and 14.16:
  * Cannot use Content-Length with non-identity Transfer-Encoding
  * Content-Range response is "bytes x-y/z" not "x-y/z"

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

net: avoid nil dereference if /etc/services can't be opened

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

syscall: fix build. WUNTRACED isn't defined for win32.
For Windows, the options for syscall.Wait4() aren't used.
Then this will be dummy value like WNOHANG, WSTOPPED.

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

crypto/cipher: fix build (missing file in Makefile)

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

arm register allocation/tempname allocation bug

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

crypto/cipher: add OFB mode

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

cc: fix vlong condition

Fixes #1032.

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

gc, runtime: make range on channel safe for multiple goroutines

Fixes #397.

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

xml: support for > in tags

This introduces support for selecting which subelement
to unmarshal into a given struct field by providing a
nesting path separated by the > character.

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

runtime: print signal information during panic

$ 6.out
panic: runtime error: invalid memory address or nil pointer dereference

[signal 11 code=0x1 addr=0x0 pc=0x1c16]

runtime.panic+0xa7 /Users/rsc/g/go/src/pkg/runtime/proc.c:1089
runtime.panic(0xf6c8, 0x25c010)
runtime.panicstring+0x69 /Users/rsc/g/go/src/pkg/runtime/runtime.c:88
runtime.panicstring(0x24814, 0x0)
runtime.sigpanic+0x144 /Users/rsc/g/go/src/pkg/runtime/darwin/thread.c:465
runtime.sigpanic()
main.f+0x16 /Users/rsc/x.go:5
main.f()
main.main+0x1c /Users/rsc/x.go:9
main.main()
runtime.mainstart+0xf /Users/rsc/g/go/src/pkg/runtime/amd64/asm.s:77
runtime.mainstart()
runtime.goexit /Users/rsc/g/go/src/pkg/runtime/proc.c:149
runtime.goexit()

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

tutorial: update go_tutorial.txt
should have been included in last CL; no idea why it was not

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

syscall: correct WSTOPPED on OS X

Have to set #defines correctly to get correct value.
Otherwise get a mask for use in implementing WIFSTOPPED(status).

Changed WSTOPPED definition in os because on
OS X WSTOPPED and WUNTRACED have different values
even though they seem to mean the same thing.

Fixes #1374.

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

tutorial: make stdin, stdout, stderr work on Windows.

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

godoc: bring back "indexing in progress" message

A wrongly nested if prevented the message from appearing.

Fixes #1420.

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

gc: fix typed constant declarations

Fixes #1411.

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

expvar: add Float

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

C: add Jos Visser (Google CLA)

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

syscall: Correct length of GNU/Linux abstract Unix domain sockaddr.

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

arm reg bug with SP adjust
after call to deferproc

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

arm bug address of a string

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

arm reg bug with address(variable)

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

disable reg - tar test failed

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

arm reg bug MOVW into bool variable

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

bytes.Buffer: fix typo in comment (doubled 'to')

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

runtime: fix arm reflect.call boundary case

The fault was lucky: when it wasn't faulting it was silently
copying a word from some other block and later putting
that same word back.  If some other goroutine had changed
that word of memory in the interim, too bad.

The ARM code was inconsistent about whether the
"argument frame" included the saved LR.  Including it made
some things more regular but mostly just caused confusion
in the places where the regularity broke.  Now the rule
reflects reality: argp is always a pointer to arguments,
never a saved link register.

Renamed struct fields to make meaning clearer.

Running ARM in QEMU, package time's gotest:
  * before: 27/58 failed
  * after: 0/50

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

cgo: In _cgo_main.c define all provided symbols as functions.

This defines crosscall2 in a way that matches that real
definition, and also defines _cgo_allocate and _cgo_panic to
indicate that they are available to be called by code compiled
with gcc.

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

runtime/cgo: Don't define crosscall2 in dummy _cgo_main.c.

In this specific package crosscall2 is already defined in a .S
file anyhow.  This avoids a warning about mismatched
alignment.

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

syscall: Correct length of SockaddrUnix.

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

gofmt, go/printer: do not insert extra linebreaks where they may break the code

Introduced a printer mode (pmode) type and corresponding
pmode values which permit easy toggling of the current
printer mode for fine-tuning of layout.

Use the printer mode to disable potential introduction of
line breaks before a closing '}' in composite literals.

Added extra test case. Applied gofmt to src and misc.

Fixes #1365.

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

fix tst instruction on arm to set overflow bit

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

godoc: remove tab before <pre>-formatted section

Fixes #1321.

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

fix build (this time for real): adjust typechecker test, ogle

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

fix build (submitted a modified parser.go file by mistake)

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

go/scanner: Make Init take a *token.File instead of a *token.FileSet.

Until now, each scan of a file added a new file to the file set.
With this change, a file can be re-scanned using the same *token.File
w/o changing the file set. Eventually this will enable the re-use of
cached source code in godoc (for the fulltext index). At the moment,
source files are read over and over again from disk.

This is the first step in that direction.

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

cmath test: fix format (%g does not print booleans)

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

add TST op code

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

CONTRIBUTORS: fix bradfitz entry

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

CONTRIBUTORS: add bradfitz@golang.org

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

http: fix scheme-relative URL parsing; add ParseRequestURL

Also adds some tests for Issue 900 which was the reason
the current URL parsing is broken.  (the previous fix
was wrong)

R=rsc, adg, dangabrad, bradfitzwork
CC=golang-dev
https://golang.org/cl/3910042

godoc: don't double HTML-escape search result snippets

Fixes #1412.

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

http: handle HEAD requests correctly

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

go_spec: specify that int and uint have the same size

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

godoc: peephole optimization for generated HTML

When searching for regular expressions such as
".", there are many consecutive matches.
In the generated HTML, combine them instead of
generating a new <span> for each adjacent text
segment highlighting a match.

Massively reduces the size of the generated
HTML in those cases.

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

tag release.2011-01-12

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

release.2011-01-12

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

disable arm registifier for the moment

R=ken2, ken3
CC=golang-dev
https://golang.org/cl/3970041

rpc: export names in debug service so it works with template changes

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

goinstall: Fix template to use exported fields

It's currently broken due to f6d5bac536c5.

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

encoding/line: fix error call (missing argument)

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

template: give "unexported" error when accessing an unexported field.

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

suffixarray: faster creation algorithm

This implements the algorithm qsufsort using the sort package
as a sorting primitive. Its worst-case performance is O(N*log(N)), and it
uses only an additional slice of N ints of memory during creation.

Benchmarks (seconds):
           old    new
10k nulls          149    0.044
1M English corpus  32.0   3.6

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

C&A for Eric Eisner
there is something funny in the database; trying to recreate the change.

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

godoc: remove TODO (bug257.go can now be indexed)

Pending CL 3752044.

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

hg: add cgo/_cgo_* to .hgignore.

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

net: use closesocket api instead of CloseHandle on Windows

thanks to piotrnar for the original CL.
Fixes #1371.

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

gob: make (en|dec)code(Ui|I)nt methods rather than functions.
No functional (ha!) change.

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

crypto/openpgp: add error and armor

error is needed by all the OpenPGP packages as they return a shared
family of error types.

armor implements OpenPGP armoring. It's very like PEM except:
  a) it includes a CRC24 checksum
  b) PEM values are small (a few KB) and so encoding/pem assumes that
     they fit in memory. Armored data can be very large and so this
     package presents a streaming interface.

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

crypto: add twofish package

The code was (as annotated in the source code) adopted from Tom's
LibCrypt (public domain code, [1]). It was neither optimised for speed
nor were any security features added.

[1] http://libtom.org/?page=features&newsitems=5&whatfile=crypt

R=rsc, berengarlehr, albert.strasheim, agl1
CC=golang-dev, schulze
https://golang.org/cl/2687042

turn off peep half of optimizer for release

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

json: do not Marshal unexported struct fields

R=r, cw, niemeyer, rsc
CC=golang-dev
https://golang.org/cl/3952041

runtime: revert 6974:1f3c3696babb

I missed that environment is used during runtime setup,
well before go init() functions run. Implemented os-dependent
runtime.goenvs functions to allow for different unix, plan9 and
windows versions of environment discovery.

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

os: add Sync to *File, wraps syscall.Fsync

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

template: allow accesses only to exported fields and methods

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

ld: Fix exported dynamic symbols on 386 Mach-O.

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

http: fix text displayed in Redirect

In the case where r.Method == "POST", was
calling Printf with an argument but "" format string,
causing a spurious %!EXTRA(...) message.

Also escape string properly in HTML generation.

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

ld: Fix exported dynamic symbols on Mach-O.

* Avoid confusion between imported and exported symbols.
* Record number of imported and exported symbols correctly.
* Explictly relocate SMACHOSYM section, since it is not in datap.

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

gob: do not encode or decode unexported fields
Such fields are simply ignored.

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

netchan: make fields public for pending gob change

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