Commits · f79bcb4b86f9a9638e7574d37732ac95db890b67 · go / golang

27 Jun, 2011 6 commits

codereview: make --ignore_hgpatch_failure work again · f79bcb4b
Russ Cox authored Jun 27, 2011
```
R=golang-dev, bradfitz
CC=golang-dev
https://golang.org/cl/4627067
```
f79bcb4b

http: do TLS handshake explicitly before copying TLS state · 7e29f1ad

Brad Fitzpatrick authored Jun 27, 2011

Previously we were snapshotting the TLS state into *Request
before we did the HTTP ReadRequest, the first Read of which
triggered the TLS handshake implicitly.

Fixes #1956

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

7e29f1ad

gotest: add -test.benchtime and -test.cpu flags. · 9843ca5e

Dmitriy Vyukov authored Jun 27, 2011

-test.benchtime allows to specify benchmark execution time.
-test.cpu allows to execute tests/benchmarks for several
values of GOMAXPROCS.

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

9843ca5e

5l, 6l, 8l: drop use of ed during build · 9dd354c6

Russ Cox authored Jun 27, 2011

build runs with chmod 0 /bin/ed now

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

9dd354c6

syscall: regenerate zerrors for darwin/linux/freebsd · dfacfd64

Russ Cox authored Jun 27, 2011

did darwin on mac with older, not broken xcode.
did linux arm by copying diffs from linux 386.
did freebsd amd64 by copying diffs from freebsd 386.

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

dfacfd64

crypto: replace "crypto/block" with "crypto/cipher" in comments · d1dbfe58

Dmitry Chestnykh authored Jun 27, 2011

Documentation mentioned the obsolete package "crypto/block",
which has been replaced with "crypto/cipher".

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

d1dbfe58

26 Jun, 2011 1 commit
- gofix: fixes for path/filepath changes · da19831c
  Robert Hencke authored Jun 26, 2011
```
Fixes #1970.

R=rsc, r
CC=golang-dev
https://golang.org/cl/4650050
```
  da19831c
25 Jun, 2011 3 commits

gofix: fixes for sort changes · 69905c62
Robert Hencke authored Jun 25, 2011
```
Fixes #1969.

R=gri, rsc, r
CC=golang-dev
https://golang.org/cl/4634076
```
69905c62

ebnflint: better handling of stdin · f70e8ed0

Robert Griesemer authored Jun 25, 2011

- don't rely on /dev/stdin as the name for standard input
- employ EBNF extraction if the source contains tags

"cat source.html | ebnflint" works now

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

f70e8ed0

fmt: Added SkipSpace() function to fmt's ScanState interface. · d94e350f

Michael T. Jones authored Jun 25, 2011

Users of the Scan() infrastructure that employ ReadRune() rather than
Token() need a way to skip leading spaces and newlines as set by the
the parent, Fscan(), Fscanln, or Fscanf(). As the internal methods and
boolean flags are not exported, this new function was added here and
in the Int and Nat Scan() functions of the big package. (fmt.Rat did
not need change since it uses Token()) Also added Printf style format
code support to int types and tests for same to int_test.go

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

d94e350f

24 Jun, 2011 12 commits

http: better handling of 0-length Request.Body · 5d4eea6a

Brad Fitzpatrick authored Jun 24, 2011

As rsc suggested after change 58a6bdac3d12 was committed, we
now read the first byte of Request.Body when the
Request.ContentLength is 0 to disambiguate between a truly
zero-length body and a body of unknown length where the user
didn't set the ContentLength field.

This was also causing the reverse proxy problem where incoming
requests (which always have a body, of private type http.body,
even for 0-lengthed requests) were being relayed to the http
Transport for fetching, which was serializing the request as a
chunked request (since ContentLength was 0 and Body was
non-nil)

Fixes #1999

R=golang-dev, kevlar
CC=golang-dev
https://golang.org/cl/4628063

5d4eea6a

http: assume ContentLength 0 on GET requests · e16b7407

Brad Fitzpatrick authored Jun 24, 2011

Incremental step in fix for issue 1999

R=golang-dev, kevlar
CC=golang-dev
https://golang.org/cl/4667041

e16b7407

mime: lower-case media type parameters · e0e28d2b

Pascal S. de Kloe authored Jun 24, 2011

        RFC 1521 section 4 states "The type, subtype, and parameter names are not case sensitive.".

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

e0e28d2b

os: remove duplicate package description · 7a471ead

Robert Hencke authored Jun 24, 2011

file.go contains a more complete package description.

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

7a471ead

misc/emacs: update list of builtins. · fc57c4b7
Quan Yong Zhai authored Jun 24, 2011
```
R=golang-dev, gri
CC=golang-dev
https://golang.org/cl/4630065
```
fc57c4b7
path/filepath: enable TestWalk to run on windows · a39ead7e
Alex Brainman authored Jun 24, 2011
```
R=golang-dev
CC=golang-dev
https://golang.org/cl/4630066
```
a39ead7e
tag weekly.2011-06-23 · 163f70ff
Andrew Gerrand authored Jun 24, 2011
```
R=r
CC=golang-dev
https://golang.org/cl/4635067
```
163f70ff

weekly.2011-06-23 · 30d08b40

Andrew Gerrand authored Jun 24, 2011

R=golang-dev, robert.hencke, r
CC=golang-dev
https://golang.org/cl/4625062

30d08b40

os: fixed PathListSeparator to ';' for windows. · 8ab2c989
Yasuhiro Matsumoto authored Jun 24, 2011
```
Fixed issue 1992

R=golang-dev, alex.brainman
CC=golang-dev
https://golang.org/cl/4661047
```
8ab2c989
http: buffer Request.Write · 89873e60
Brad Fitzpatrick authored Jun 24, 2011
```
Fixes #1996

R=golang-dev, r
CC=golang-dev
https://golang.org/cl/4639068
```
89873e60

runtime: don't use twice the memory with grsec-like kernels · 65b036c3

Gustavo Niemeyer authored Jun 24, 2011

grsec needs the FIXED flag to be provided to mmap, which
works now.  That said, when the allocation fails to be made
in the specific address, we're still given back a writable
page.  This change will unmap that page to avoid using
twice the amount of memory needed.

It'd also be pretty easy to avoid the extra system calls
once we detected that the flag is needed, but I'm not sure
if that edge case is worth the effort.

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

65b036c3

goinstall: build with make by default, add -make flag · b4bab6f8

Andrew Gerrand authored Jun 24, 2011

This is a temporary measure until go/build can build cgo packages.

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

b4bab6f8

23 Jun, 2011 7 commits

mime/multipart: remove newline at top of the multipart. · 73d741fd
Yasuhiro Matsumoto authored Jun 23, 2011
```
R=golang-dev, bradfitz, arctanofyourface
CC=golang-dev
https://golang.org/cl/4635063
```
73d741fd

libmach: fix disassembly of FCMOVcc and FCOMI · d8474670

Anthony Martin authored Jun 23, 2011

The optable for 0xDB is handled specially.

This was the cause of a really weird bug
when using cov (386!) on the math package.

A real head-scratcher.

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

d8474670

libmach: fix tracing on linux (for cov) · 8f72a87b
Anthony Martin authored Jun 23, 2011
```
R=rsc, dave
CC=golang-dev
https://golang.org/cl/4629064
```
8f72a87b

5c: do not use R9 and R10 · c475c307

Russ Cox authored Jun 23, 2011

This program used to use R9 and R10.
Now it fails to compile (out of registers).
I used to know a simpler test but can't remember it.

Learned something new: Rietveld refuses change
list descriptions bigger than 10 kB.

int sum(int x, int y, int z, int w) {
        return
        (((((((x*y+z*w|x*y+z*w)^
        (x*y+z*w|x*y+z*w))&
        ((x*y+z*w|x*y+z*w)^
        (x*y+z*w|x*y+z*w)))|
        (((x*y+z*w|x*y+z*w)^
        (x*y+z*w|x*y+z*w))&
        ((x*y+z*w|x*y+z*w)^
        (x*y+z*w|x*y+z*w))))/
        ((((x*y+z*w|x*y+z*w)^
        (x*y+z*w|x*y+z*w))&
        ((x*y+z*w|x*y+z*w)^
        (x*y+z*w|x*y+z*w)))|
        (((x*y+z*w|x*y+z*w)^
        (x*y+z*w|x*y+z*w))&
        ((x*y+z*w|x*y+z*w)^
        (x*y+z*w|x*y+z*w)))))%
        (((((x*y+z*w|x*y+z*w)^
        (x*y+z*w|x*y+z*w))&
        ((x*y+z*w|x*y+z*w)^
        (x*y+z*w|x*y+z*w)))|
        (((x*y+z*w|x*y+z*w)^
        (x*y+z*w|x*y+z*w))&
        ((x*y+z*w|x*y+z*w)^
        (x*y+z*w|x*y+z*w))))/
        ((((x*y+z*w|x*y+z*w)^
        (x*y+z*w|x*y+z*w))&
        ((x*y+z*w|x*y+z*w)^
        (x*y+z*w|x*y+z*w)))|
        (((x*y+z*w|x*y+z*w)^
        (x*y+z*w|x*y+z*w))&
        ((x*y+z*w|x*y+z*w)^
        (x*y+z*w|x*y+z*w))))))*
        ((((((x*y+z*w|x*y+z*w)^
        (x*y+z*w|x*y+z*w))&
        ((x*y+z*w|x*y+z*w)^
        (x*y+z*w|x*y+z*w)))|
        (((x*y+z*w|x*y+z*w)^
        (x*y+z*w|x*y+z*w))&
        ((x*y+z*w|x*y+z*w)^
        (x*y+z*w|x*y+z*w))))/
        ((((x*y+z*w|x*y+z*w)^
        (x*y+z*w|x*y+z*w))&
        ((x*y+z*w|x*y+z*w)^
        (x*y+z*w|x*y+z*w)))|
        (((x*y+z*w|x*y+z*w)^
        (x*y+z*w|x*y+z*w))&
        ((x*y+z*w|x*y+z*w)^
        (x*y+z*w|x*y+z*w)))))%
        (((((x*y+z*w|x*y+z*w)^
        (x*y+z*w|x*y+z*w))&
        ((x*y+z*w|x*y+z*w)^
        (x*y+z*w|x*y+z*w)))|
        (((x*y+z*w|x*y+z*w)^
        (x*y+z*w|x*y+z*w))&
        ((x*y+z*w|x*y+z*w)^
        (x*y+z*w|x*y+z*w))))/
        ((((x*y+z*w|x*y+z*w)^
        (x*y+z*w|x*y+z*w))&
        ((x*y+z*w|x*y+z*w)^
        (x*y+z*w|x*y+z*w)))|
        (((x*y+z*w|x*y+z*w)^
        (x*y+z*w|x*y+z*w))&
        ((x*y+z*w|x*y+z*w)^
        (x*y+z*w|x*y+z*w)))))))*
        (((((((x*y+z*w|x*y+z*w)^
        (x*y+z*w|x*y+z*w))&
        ((x*y+z*w|x*y+z*w)^
        (x*y+z*w|x*y+z*w)))|
        (((x*y+z*w|x*y+z*w)^
        (x*y+z*w|x*y+z*w))&
        ((x*y+z*w|x*y+z*w)^
        (x*y+z*w|x*y+z*w))))/
        ((((x*y+z*w|x*y+z*w)^
        (x*y+z*w|x*y+z*w))&
        ((x*y+z*w|x*y+z*w)^
        (x*y+z*w|x*y+z*w)))|
        (((x*y+z*w|x*y+z*w)^
        (x*y+z*w|x*y+z*w))&
        ((x*y+z*w|x*y+z*w)^
        (x*y+z*w|x*y+z*w)))))%
        (((((x*y+z*w|x*y+z*w)^
        (x*y+z*w|x*y+z*w))&
        ((x*y+z*w|x*y+z*w)^
        (x*y+z*w|x*y+z*w)))|
        (((x*y+z*w|x*y+z*w)^
        (x*y+z*w|x*y+z*w))&
        ((x*y+z*w|x*y+z*w)^
        (x*y+z*w|x*y+z*w))))/
        ((((x*y+z*w|x*y+z*w)^
        (x*y+z*w|x*y+z*w))&
        ((x*y+z*w|x*y+z*w)^
        (x*y+z*w|x*y+z*w)))|
        (((x*y+z*w|x*y+z*w)^
        (x*y+z*w|x*y+z*w))&
        ((x*y+z*w|x*y+z*w)^
        (x*y+z*w|x*y+z*w))))))*
        ((((((x*y+z*w|x*y+z*w)^
        (x*y+z*w|x*y+z*w))&
        ((x*y+z*w|x*y+z*w)^
        (x*y+z*w|x*y+z*w)))|
        (((x*y+z*w|x*y+z*w)^
        (x*y+z*w|x*y+z*w))&
        ((x*y+z*w|x*y+z*w)^
        (x*y+z*w|x*y+z*w))))/
        ((((x*y+z*w|x*y+z*w)^
        (x*y+z*w|x*y+z*w))&
        ((x*y+z*w|x*y+z*w)^
        (x*y+z*w|x*y+z*w)))|
        (((x*y+z*w|x*y+z*w)^
        (x*y+z*w|x*y+z*w))&
        ((x*y+z*w|x*y+z*w)^
        (x*y+z*w|x*y+z*w)))))%
        (((((x*y+z*w|x*y+z*w)^
        (x*y+z*w|x*y+z*w))&
        ((x*y+z*w|x*y+z*w)^
        (x*y+z*w|x*y+z*w)))|
        (((x*y+z*w|x*y+z*w)^
        (x*y+z*w|x*y+z*w))&
        ((x*y+z*w|x*y+z*w)^
        (x*y+z*w|x*y+z*w))))/
        ((((x*y+z*w|x*y+z*w)^
        (x*y+z*w|x*y+z*w))&
        ((x*y+z*w|x*y+z*w)^
        (x*y+z*w|x*y+z*w)))|
        (((x*y+z*w|x*y+z*w)^
        (x*y+z*w|x*y+z*w))&
        ((x*y+z*w|x*y+z*w)^
        (x*y+z*w|x*y+z*w)))))))
        ;
}

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

c475c307

exec: LookPath should not search %PATH% for files like c:cmd.exe · 5d53aab4
Alex Brainman authored Jun 23, 2011
```
R=golang-dev
CC=golang-dev
https://golang.org/cl/4632055
```
5d53aab4

exp/template: make -0 be an unsigned int. · 96731619

Rob Pike authored Jun 23, 2011

Fix (inconsequential) type error in list initializer.

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

96731619

go/build: include Import objects in Script Inputs · d7df79a7

Andrew Gerrand authored Jun 23, 2011

This has the effect of making goinstall rebuild a package's
dependencies when they are newer than the current package object.

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

d7df79a7

22 Jun, 2011 11 commits

net, syscall: interface for windows · 88442358

Yasuhiro Matsumoto authored Jun 22, 2011

R=mikioh.mikioh, alex.brainman, rsc, vincent.vanackere
CC=golang-dev
https://golang.org/cl/4590050

88442358

io: clarify Read, ReadAt, Copy, Copyn EOF behavior · 36b5e1d6
Russ Cox authored Jun 22, 2011
```
R=golang-dev, bradfitz, iant, dsymonds, nigeltao, r
CC=golang-dev
https://golang.org/cl/4629062
```
36b5e1d6

First cut at the parser for the new template package. · cd7826e5

Rob Pike authored Jun 22, 2011

This is not a full grammar, but the pieces are there to implement whatever we converge on.

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

cd7826e5

exec: better error message for windows LookPath · 123549ac
Alex Brainman authored Jun 22, 2011
```
Fixes #1991.

R=golang-dev, bradfitz, rsc
CC=golang-dev
https://golang.org/cl/4648049
```
123549ac
image: basic test for the 16-bits-per-color-channel types. · 6b4fb9f8
Nigel Tao authored Jun 22, 2011
```
R=r
CC=golang-dev
https://golang.org/cl/4635059
```
6b4fb9f8

syscall: add tty support to StartProcess · c8443d72

Ken Rockot authored Jun 22, 2011

These changes add a Ctty int field to the Unix syscall.ProcAttr which,
if set >= 0 in conjuction with Setsid=true, will be used by
forkAndExecInChild as the file descriptor for the new child's
controlling terminal.

Necessary changes have been made to mkerrors.sh to generate defs for
TIOC*, though changes to its output files are not included here.
The changes made should support Linux, FreeBSD and Darwin, at least.

R=iant, bradfitz, r, rsc, borman
CC=golang-dev
https://golang.org/cl/4532075

c8443d72

A+C: Ken Rockot (individual CLA) · 8a4acd53
Russ Cox authored Jun 22, 2011
```
R=golang-dev, dsymonds
CC=golang-dev
https://golang.org/cl/4648053
```
8a4acd53
build: explain $PWD use · 2e41120a
Russ Cox authored Jun 22, 2011
```
R=golang-dev, gri
CC=golang-dev
https://golang.org/cl/4639064
```
2e41120a
sync: restore GOMAXPROCS during benchmarks · 8cab7f57
Dmitriy Vyukov authored Jun 22, 2011
```
R=golang-dev
CC=dvyukov, golang-dev
https://golang.org/cl/4631060
```
8cab7f57

ld: don't attempt to build dynamic sections unnecessarily · 89d334ff

Gustavo Niemeyer authored Jun 22, 2011

This prevents ld from generating zeroed symtab entries for
sections that aren't going to be generated because dynamic
linkage has been disabled (-d was used or no dynamic libs
were seen).  Even though they were not explicitly added by
doelf, the section creation process was making them
reachable again.

The windows head is being disconsidered for this because
apparently it's not taking into account debug['d'].

This makes elflint 0.1% happier.

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

89d334ff

Fix Windows build; ErrorString->NewError · 5dfdccf2
Brad Fitzpatrick authored Jun 22, 2011
```
R=golang-dev, gri
CC=golang-dev
https://golang.org/cl/4634080
```
5dfdccf2