Commits · b4829c1de6ffd8581c40932da7a57dcfdd0610fb · go / golang

19 Jan, 2012 20 commits

html: in foreign content, check for HTML integration points in breakout · b4829c1d

Nigel Tao authored Jan 19, 2012

elements.

Pass tests10.dat, test 33:
<!DOCTYPE html><svg><desc><svg><ul>a

| <!DOCTYPE html>
| <html>
|   <head>
|   <body>
|     <svg svg>
|       <svg desc>
|         <svg svg>
|         <ul>
|           "a"

Also pass test 34:
<!DOCTYPE html><p><svg><desc><p>

R=andybalholm, dsymonds
CC=golang-dev
https://golang.org/cl/5536048

b4829c1d

encoding/json: fix comments, tweak tests for tag names · d4fe9c6a
Mikio Hara authored Jan 19, 2012
```
R=bradfitz
CC=golang-dev
https://golang.org/cl/5558047
```
d4fe9c6a

regexp: add SubexpNames · 21d3721e

Russ Cox authored Jan 19, 2012

Fixes #2440.

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

21d3721e

syscall: fix plan9 build · e3e93b0f
Mikio Hara authored Jan 19, 2012
```
R=golang-dev, bradfitz
CC=golang-dev
https://golang.org/cl/5532097
```
e3e93b0f

godoc: remove "need more packages?" link · e547393f

Andrew Gerrand authored Jan 19, 2012

Its need will be addressed in another way by the redesign.

Fixes #2719.

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

e547393f

doc/codewalk: update urlpoll to use time.Duration. · 1de49311
Stefan Nilsson authored Jan 19, 2012
```
R=adg
CC=golang-dev
https://golang.org/cl/5545061
```
1de49311
A+C: add Stefan Nilsson's gmail account · 885cbc3b
Andrew Gerrand authored Jan 19, 2012
```
R=golang-dev, dsymonds
CC=golang-dev
https://golang.org/cl/5541064
```
885cbc3b
go/doc: collect imports · c109705c
Robert Griesemer authored Jan 19, 2012
```
R=r
CC=golang-dev
https://golang.org/cl/5556051
```
c109705c
cmd/go: skip _obj directories in package scans · f47807a5
Brad Fitzpatrick authored Jan 19, 2012
```
Fixes #2693

R=golang-dev, rsc
CC=golang-dev
https://golang.org/cl/5557057
```
f47807a5
net: fix plan9 build · b58b5ba9
Mikio Hara authored Jan 19, 2012
```
R=golang-dev, bradfitz
CC=golang-dev
https://golang.org/cl/5554058
```
b58b5ba9

net: fix comments · 2356e438

Mikio Hara authored Jan 19, 2012

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

2356e438

net/http: update the Client docs a bit · f44304ee
Brad Fitzpatrick authored Jan 19, 2012
```
R=golang-dev, dsymonds, adg
CC=golang-dev
https://golang.org/cl/5557058
```
f44304ee
syscall: add Unix method to TimeSpec, TimeVal · 7d418aee
Brad Fitzpatrick authored Jan 19, 2012
```
Fixes #2534

R=golang-dev, dave, alex.brainman
CC=golang-dev
https://golang.org/cl/5554057
```
7d418aee
encoding/json: allow / and % in tag names · b39c883e
Brad Fitzpatrick authored Jan 19, 2012
```
Fixes #2718

R=golang-dev, adg
CC=golang-dev
https://golang.org/cl/5532095
```
b39c883e
path: added examples · 0d8c6b4f
Sanjay Menakuru authored Jan 19, 2012
```
R=rsc, adg
CC=golang-dev
https://golang.org/cl/5503089
```
0d8c6b4f

image: add PixOffset methods; use them in image/draw and image/tiff. · af08cfa4

Nigel Tao authored Jan 19, 2012

image/draw benchmarks show <1% change for the fast paths.
The slow paths got worse by 1-4%, but they're the slow paths.
I don't care so much about them, and presumably compiler improvements
could claw it back.

IIUC 6g's inlining is enabled by default now.

benchmark                        old ns/op    new ns/op    delta
draw.BenchmarkFillOver             2988384      2999624   +0.38%
draw.BenchmarkFillSrc               153141       153262   +0.08%
draw.BenchmarkCopyOver             2155756      2170831   +0.70%
draw.BenchmarkCopySrc                72591        72646   +0.08%
draw.BenchmarkNRGBAOver            2487372      2491576   +0.17%
draw.BenchmarkNRGBASrc             1361306      1409180   +3.52%
draw.BenchmarkYCbCr                2540712      2562359   +0.85%
draw.BenchmarkGlyphOver            1004879      1023308   +1.83%
draw.BenchmarkRGBA                 8746670      8844455   +1.12%
draw.BenchmarkGenericOver         31860960     32512960   +2.05%
draw.BenchmarkGenericMaskOver     16369060     16435720   +0.41%
draw.BenchmarkGenericSrc          13128540     13127810   -0.01%
draw.BenchmarkGenericMaskSrc      30059300     28883210   -3.91%

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

af08cfa4

net: use NewTimer, not NewTicker, in fd_windows.go · 98af3880

Brad Fitzpatrick authored Jan 19, 2012

It works with NewTicker too, but is potentially a bit less efficient,
and reads wrong.

This is what happens when you TBR Windows changes, I guess.

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

98af3880

godoc: specify HTML page metadata with a JSON blob · 7cb21a79

Andrew Gerrand authored Jan 19, 2012

This allows HTML pages to specify arbitrary data in a header:

<!--{
        "Title": "The page title",
        ...
}-->

replacing the old style comments:

<!-- title The page title -->

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

7cb21a79

net: change SetTimeout to SetDeadline · b71883e9

Brad Fitzpatrick authored Jan 19, 2012

Previously, a timeout (in int64 nanoseconds) applied to a granularity
even smaller than one operation:  a 100 byte read with a 1 second timeout
could take 100 seconds, if the bytes all arrived on the network 1 second
apart.  This was confusing.

Rather than making the timeout granularity be per-Read/Write,
this CL makes callers set an absolute deadline (in time.Time)
after which operations will fail.  This makes it possible to
set deadlines at higher levels, without knowing exactly how
many read/write operations will happen in e.g. reading an HTTP
request.

Fixes #2723

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

b71883e9

test: split golden.out into expected output per test · 5e77b009
Ian Lance Taylor authored Jan 19, 2012
```
This will permit gccgo to check test output.

R=golang-dev, r
CC=golang-dev
https://golang.org/cl/5554056
```
5e77b009

18 Jan, 2012 20 commits

fix windows build: always use / in filenames of go/doc test output · eaf8295f
Robert Griesemer authored Jan 18, 2012
```
R=r
CC=golang-dev
https://golang.org/cl/5554055
```
eaf8295f
doc/go1.*: documented changes to go/* package tree · 0203fbee
Robert Griesemer authored Jan 18, 2012
```
R=r
CC=golang-dev
https://golang.org/cl/5557053
```
0203fbee

image/color: rename modelYCbCr to yCbCrModel. · be93207c

Benny Siegert authored Jan 18, 2012

This matches the other color models. It seems that this was missed
during the review of 5544073.

R=nigeltao, david.crawshaw
CC=golang-dev
https://golang.org/cl/5536057

be93207c

test: change several tests to not print · f2030938

Ian Lance Taylor authored Jan 18, 2012

This will make these tests more meaningful for gccgo, which
runs tests in parallel and has no equivalent to golden.out.

Remove ken/simpprint.go since it duplicates helloworld.go.

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

f2030938

go/doc: rewrote and completed test framework · 39bb4bd4

Robert Griesemer authored Jan 18, 2012

Packages to test are kept in ./testdata together
with the corresponding golden (packagename.out)
file.

To update the golden files, run: go test -update

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

39bb4bd4

go/token: replaced Files() with Iterate() · 9edabbe0

Robert Griesemer authored Jan 18, 2012

- Use a faster closure-based iterator rather than a channel-based one.
- Otherwise: better code organization, but no other API changes.

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

9edabbe0

test: gofmt a few tests · 6b346282

Ian Lance Taylor authored Jan 18, 2012

I'm planning to change these tests, but the gofmt changes are
fairly extensive, so I'm separating the gofmt changes from the
substantive changes.

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

6b346282

time: add Since, which returns the time elapsed since some past time t. · 20812c49
Sameer Ajmani authored Jan 18, 2012
```
R=rsc, r
CC=golang-dev
https://golang.org/cl/5532088
```
20812c49

exp/ssh: handle versions with just '\n' · dbebb086

Adam Langley authored Jan 18, 2012

djm recommend that we do this because OpenSSL was only fixed in 2008:
http://anoncvs.mindrot.org/index.cgi/openssh/sshd.c?revision=1.380&view=markup

R=dave, jonathan.mark.pittman
CC=golang-dev
https://golang.org/cl/5555044

dbebb086

crypto/tls: add FreeBSD root certificate location · 18de1147
Shenghou Ma authored Jan 18, 2012
```
    Fixes #2721.

R=golang-dev, bradfitz
CC=golang-dev
https://golang.org/cl/5532090
```
18de1147
pkg/bytes: add the usual copyright notice to example_test.go · dcc5fe1e
Olivier Duperray authored Jan 18, 2012
```
R=golang-dev, gri
CC=golang-dev
https://golang.org/cl/5553050
```
dcc5fe1e
go/scanner: fix example · 90c38c46
Olivier Duperray authored Jan 18, 2012
```
R=golang-dev, gri
CC=golang-dev
https://golang.org/cl/5532085
```
90c38c46

gc: fieldnames in structliterals in exported inlines should not be qualified if… · 6ff01f01

Luuk van Dijk authored Jan 18, 2012

gc: fieldnames in structliterals in exported inlines should not be qualified if they're embedded builtin types.

Trust me.
Fixes #2687.

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

6ff01f01

doc/go1.html: add notes about hash.Hash · 1f1c9baf
Russ Cox authored Jan 18, 2012
```
R=golang-dev, r
CC=golang-dev
https://golang.org/cl/5536053
```
1f1c9baf

crypto/hmac: Add HMAC-SHA224 and HMAC-SHA384/512 · a5263c7c

Luit van Drongelen authored Jan 18, 2012

First was, apart from adding tests, a single line of code (to add the
constructor function). Adding SHA512-based hashing to crypto/hmac
required minor rework of the package because of a previously hardcoded
block-size in it's implementation. Instead of using a hash.Hash
generator function the constructor function now uses a crypto.Hash
type, which was extended to expose information about block size.

The only standard library package impacted by the change is
crypto/tls, for which the fix is included in this patch. It might be
useful to extend gofix to include this API change too.

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

a5263c7c

gc: handle printing of string/arrayrune conversions · 18ee75ec
Luuk van Dijk authored Jan 18, 2012
```
Fixes #2714.

R=rsc
CC=golang-dev
https://golang.org/cl/5540066
```
18ee75ec
os: remove old note about NewSyscallError being special · 436b37d8
Alex Brainman authored Jan 18, 2012
```
R=golang-dev, r
CC=golang-dev
https://golang.org/cl/5556044
```
436b37d8

test: match gccgo error messages for bug345 · 6a5db20d

Ian Lance Taylor authored Jan 18, 2012

bug345.dir/main.go:25:18: error: argument 1 has incompatible type (need explicit conversion; missing method ‘Write’)
bug345.dir/main.go:27:8: error: argument 1 has incompatible type

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

6a5db20d

test: fix bug364 to actually run · 24c76f68
Ian Lance Taylor authored Jan 18, 2012
```
R=golang-dev, gri
CC=golang-dev
https://golang.org/cl/5536044
```
24c76f68
encoding/json: document angle bracket escaping. · 6e285eba
David Symonds authored Jan 18, 2012
```
Fixes #2643.

R=rsc, d_smithson
CC=golang-dev
https://golang.org/cl/5543068
```
6e285eba