Commits · ede44c68a48ba579defa08d1df94b268c93ab8e3 · Kirill Smelkov / go

02 Nov, 2011 25 commits

weekly.2011-11-02 · ede44c68

Andrew Gerrand authored Nov 02, 2011

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

ede44c68

all: rename os.Error to error in various non-code contexts · 492098eb
Russ Cox authored Nov 01, 2011
```
R=adg
CC=golang-dev
https://golang.org/cl/5328062
```
492098eb

os: fixes for error (plan9) · abd32609

Russ Cox authored Nov 01, 2011

The Plan 9 build stops in runtime,
but might as well fix these anyway.

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

abd32609

net: update for error (linux) · f7b7338e
Russ Cox authored Nov 01, 2011
```
R=adg
CC=golang-dev
https://golang.org/cl/5303091
```
f7b7338e
os, syscall: update for error · f1b64aa7
Russ Cox authored Nov 01, 2011
```
R=adg
CC=golang-dev
https://golang.org/cl/5333052
```
f1b64aa7
non-pkg: gofix -r error -force=error · 44526cdb
Russ Cox authored Nov 01, 2011
```
R=golang-dev, iant, r, r
CC=golang-dev
https://golang.org/cl/5307066
```
44526cdb
src/pkg/[n-z]*: gofix -r error -force=error · eb692929
Russ Cox authored Nov 01, 2011
```
R=golang-dev, bsiegert, iant
CC=golang-dev
https://golang.org/cl/5294074
```
eb692929
src/pkg/[a-m]*: gofix -r error -force=error · c2049d2d
Russ Cox authored Nov 01, 2011
```
R=golang-dev, iant
CC=golang-dev
https://golang.org/cl/5322051
```
c2049d2d
tutorial,effective_go: prepare for error change · 68050ac7
Rob Pike authored Nov 01, 2011
```
R=adg, rsc
CC=golang-dev
https://golang.org/cl/5316068
```
68050ac7

exec: introduce ExitError · 451a1fa4

Russ Cox authored Nov 01, 2011

The existing code uses *os.Waitmsg as an os.Error,
but *os.Waitmsg is really just a stringer.

Introduce an explicit error type for the real error.

Not to be submitted until just before error goes in;
the gofix for error updates type assertions
        err.(*os.Waitmsg)
to
        err.(*exec.ExitError)

The seemingly redundant String method will become
an Error method when error goes in, and will no longer
be redundant.

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

451a1fa4

exp/ebnf: manual fixup for error · c93b6a17

Russ Cox authored Nov 01, 2011

(The definition of ErrorList is in another file, so gofix
has no hope of getting this right.)

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

c93b6a17

cgo, gotest: use error instead of os.Error in generated code · c8ad1a4d
Russ Cox authored Nov 01, 2011
```
R=golang-dev, iant
CC=golang-dev
https://golang.org/cl/5319057
```
c8ad1a4d
os: use error, io.EOF · 08a073a1
Russ Cox authored Nov 01, 2011
```
R=r
CC=golang-dev
https://golang.org/cl/5298073
```
08a073a1
io: use error, add EOF, avoid os · c06cf03f
Russ Cox authored Nov 01, 2011
```
R=r, r
CC=golang-dev
https://golang.org/cl/5311068
```
c06cf03f
runtime: update for error · c14f71c7
Russ Cox authored Nov 01, 2011
```
R=golang-dev, iant, r
CC=golang-dev
https://golang.org/cl/5306075
```
c14f71c7
exp/types: add error type to universe · e67d3c44
Russ Cox authored Nov 01, 2011
```
R=golang-dev, iant
CC=golang-dev
https://golang.org/cl/5327051
```
e67d3c44
misc/vim: add error type. · 782fd1fc
David Symonds authored Nov 01, 2011
```
R=golang-dev, rsc
CC=golang-dev
https://golang.org/cl/5331046
```
782fd1fc

errors: new package · e4ae30f5

Russ Cox authored Nov 01, 2011

The only function is errors.New, at least for now.

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

e4ae30f5

gc: add error type · 2a0e15d3
Russ Cox authored Nov 01, 2011
```
R=ken
CC=golang-dev
https://golang.org/cl/5331043
```
2a0e15d3
http: avoid name error in test · b4e35629
Russ Cox authored Nov 01, 2011
```
R=adg
CC=golang-dev
https://golang.org/cl/5316070
```
b4e35629

gofix: error fix · 758200f2

Russ Cox authored Nov 01, 2011

To make the error fix more useful, expand typecheck to gather
more information about struct fields, typecheck range statements,
typecheck indirect and index of named types, and collect information
about assignment conversions.

Also, change addImport to rename top-level uses of a to-be-imported
identifier to avoid conflicts.  This duplicated some of the code in
the url fix, so that fix is now shorter.

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

758200f2

spec: add error · d9877e22

Russ Cox authored Nov 01, 2011

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

d9877e22

tag weekly.2011-11-01 · 9a056354
Andrew Gerrand authored Nov 02, 2011
```
R=golang-dev, rsc, dsymonds
CC=golang-dev
https://golang.org/cl/5305085
```
9a056354
weekly.2011-11-01 · 08757f72
Andrew Gerrand authored Nov 02, 2011
```
R=rsc, r
CC=golang-dev
https://golang.org/cl/5336044
```
08757f72

html: stop at scope marker node when generating implied </a> tags · 22ee5ae2

Andrew Balholm authored Nov 02, 2011

A <a> tag generates implied end tags for any open <a> elements.
But it shouldn't do that when it is inside a table cell the the open <a>
is outside the table.
So stop the search for an open <a> when we reach a scope marker node.

Pass tests1.dat, test 78:
<a href="blah">aba<table><tr><td><a href="foo">br</td></tr>x</table>aoe

| <html>
|   <head>
|   <body>
|     <a>
|       href="blah"
|       "abax"
|       <table>
|         <tbody>
|           <tr>
|             <td>
|               <a>
|                 href="foo"
|                 "br"
|       "aoe"

Also pass test 79:
<table><a href="blah">aba<tr><td><a href="foo">br</td></tr>x</table>aoe

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

22ee5ae2

01 Nov, 2011 11 commits

html: refactor the blacklist for the "render and re-parse" test. · 90b76c0f
Nigel Tao authored Nov 02, 2011
```
R=andybalholm
CC=golang-dev, mikesamuel
https://golang.org/cl/5331056
```
90b76c0f

textproto: prevent long lines in HTTP headers from causing HTTP 400 responses. · f753e3fa

Mike Solomon authored Nov 01, 2011

This fixes the issue without an extra copy in the average case.

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

f753e3fa

doc/Makefile: add 'all' rule to build all docs · cf7281e7
Rob Pike authored Nov 01, 2011
```
R=golang-dev, rsc
CC=golang-dev
https://golang.org/cl/5334044
```
cf7281e7
gc: test + fix escape analysis bug · b4df33a6
Russ Cox authored Nov 01, 2011
```
R=lvd
CC=golang-dev
https://golang.org/cl/5333049
```
b4df33a6
doc: refer to tour.golang.org instead of go-tour.appspot.com · 1fe22d2d
Andrew Gerrand authored Nov 01, 2011
```
R=golang-dev, r
CC=golang-dev
https://golang.org/cl/5331055
```
1fe22d2d
doc: fix typo in spec example code comment · 44262d15
Charles L. Dorian authored Nov 01, 2011
```
R=r, golang-dev, adg
CC=golang-dev
https://golang.org/cl/5308071
```
44262d15
go spec: introduce rune type · b910a273
Robert Griesemer authored Nov 01, 2011
```
R=r, iant, rsc, r
CC=golang-dev
https://golang.org/cl/5293048
```
b910a273

rpc: avoid infinite loop on input error · 2e79e8e5

Russ Cox authored Nov 01, 2011

Fixes #1828.
Fixes #2179.

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

2e79e8e5

gopack: change archive file name length back to 16 · 7b04471d

Russ Cox authored Nov 01, 2011

This CL grew the archive file name length from 16 to 64:

        changeset:   909:58574851d792
        user:        Russ Cox <rsc@golang.org>
        date:        Mon Oct 20 13:53:56 2008 -0700

Back then, every x.go file in a package became an x.6 file
in the archive.  It was important to be able to allow the
use of long Go source file names, hence the increase in size.

Today, all Go source files compile into a single _go_.6 file
regardless of their names, so the archive file name length
no longer needs to be long.  The longer name causes some
problems on Plan 9, where the native archive format is the
same but with 16-byte names, so revert back to 16.

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

7b04471d

os: do not interpret 0-length read as EOF · 4853c517
Russ Cox authored Nov 01, 2011
```
Fixes #2402.

R=golang-dev, bradfitz, r
CC=golang-dev
https://golang.org/cl/5298081
```
4853c517

html: process </td> tags; foster parent at most one node per token · 9db3f78c

Andrew Balholm authored Nov 01, 2011

Correctly close table cell when </td> is read.

Because of reconstructing the active formatting elements, more than one
node may be created when reading a single token.
If both nodes are foster parented, they will be siblings, but the first
node should be the parent of the second.

Pass tests1.dat, test 77:
<a href="blah">aba<table><a href="foo">br<tr><td></td></tr>x</table>aoe

| <html>
|   <head>
|   <body>
|     <a>
|       href="blah"
|       "aba"
|       <a>
|         href="foo"
|         "br"
|       <a>
|         href="foo"
|         "x"
|       <table>
|         <tbody>
|           <tr>
|             <td>
|     <a>
|       href="foo"
|       "aoe"

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

9db3f78c

31 Oct, 2011 4 commits

template: fix error checking on execute without parse · cae23f03

Scott Lawrence authored Oct 31, 2011

Fixed error checking in exec.go to give a sensible error message when
execution is attempted before a successful parse (rather than an
outright panic).

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

cae23f03

pkg: minor cleanup · 92926f54

Russ Cox authored Oct 31, 2011

remove some redundant .String()
change variable name to make it os.Error-proof

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

92926f54

test: make bug107 os.Error-proof · a07841e2
Russ Cox authored Oct 31, 2011
```
R=golang-dev, bradfitz
CC=golang-dev
https://golang.org/cl/5306087
```
a07841e2
runtime/pprof: document OS X being broken · d0eaa58b
Russ Cox authored Oct 31, 2011
```
R=golang-dev, r
CC=golang-dev
https://golang.org/cl/5307078
```
d0eaa58b