Commits · b910a2739629f10eea56c44467f99263ef303f46 · Kirill Smelkov / go

01 Nov, 2011 5 commits

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 13 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
time: add RFC1123 with numeric timezone format · a7e473be
Scott Lawrence authored Oct 31, 2011
```
Fixes #841.

R=rsc, r
CC=golang-dev
https://golang.org/cl/5322058
```
a7e473be
math: Improved accuracy for Sin and Cos. · 739c442e
Charles L. Dorian authored Oct 31, 2011
```
Fixes #1564.

R=rsc, dchest
CC=golang-dev
https://golang.org/cl/5320056
```
739c442e

json: Properly handle nil slices. · 48c75c5f

Alexander Reece authored Oct 31, 2011

Marshal nil slices as null and parse null value as a nil slice.
Fixes #2278.

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

48c75c5f

A+C: Alexander Reece (individual CLA) · 288dacd0
Russ Cox authored Oct 31, 2011
```
R=golang-dev, r
CC=golang-dev
https://golang.org/cl/5297072
```
288dacd0

syscall: fix Await msg on Plan 9 · d1f48db1

Andrey Mirtchovski authored Oct 31, 2011

Plan 9's await() returns '' for nil exit status but programs, most notably gotest,
see this as an error return.

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

d1f48db1

gc: clean up printing. · 50110c9f

Luuk van Dijk authored Oct 31, 2011

Got rid of all the magic mystery globals. Now
for %N, %T, and %S, the flags +,- and # set a sticky
debug, sym and export mode, only visible in the new fmt.c.
Default is error mode. Handle h and l flags consistently with
the least side effects, so we can now change
things without worrying about unrelated things
breaking.

fixes #2361

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

50110c9f

net: Plan 9 fixes · 58423360

Fazlul Shahriar authored Oct 31, 2011

Makes all tests pass.

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

58423360

misc/emacs/go-lang.el: Fix restoration of multiple windows in a frame after gofmt · 350a5ce6

Jan Newmarch authored Oct 31, 2011

If a frame has multiple windows then the windows must all be restored
after gofmt has finished and the old windows must be restored.
Before this fix, only the Go code edit window would be left.

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

350a5ce6

exp/norm: fixed bug that creeped in with moving to the new · eef78091

Marcel van Lohuizen authored Oct 31, 2011

regexp, which caused the last line of a test block to be ignored.

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

eef78091

29 Oct, 2011 1 commit

exp/ssh: fix length header leaking into channel data streams. · 0f6b80c6

Dave Cheney authored Oct 29, 2011

The payload of a data message is defined as an SSH string type,
which uses the first four bytes to encode its length. When channelData
and channelExtendedData were added I defined Payload as []byte to
be able to use it directly without a string to []byte conversion. This
resulted in the length data leaking into the payload data.

This CL fixes the bug, and restores agl's original fast path code.

Additionally, a bug whereby s.lock was not released if a packet arrived
for an invalid channel has been fixed.

Finally, as they were no longer used, I have removed
the channelData and channelExtedendData structs.

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

0f6b80c6

28 Oct, 2011 17 commits

html: adjust bookmark in "adoption agency" algorithm · 604e10c3

Andrew Balholm authored Oct 29, 2011

In the adoption agency algorithm, the formatting element is sometimes
removed from the list of active formatting elements and reinserted at a later index.
In that case, the bookmark showing where it is to be reinserted needs to be moved,
so that its position relative to its neighbors remains the same
(and also so that it doesn't become out of bounds).

Pass tests1.dat, test 70:
<DIV> abc <B> def <I> ghi <P> jkl </B>

| <html>
|   <head>
|   <body>
|     <div>
|       " abc "
|       <b>
|         " def "
|         <i>
|           " ghi "
|       <i>
|         <p>
|           <b>
|             " jkl "

Also pass tests through test 76:
<test attribute---------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------->

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

604e10c3

html: don't run "adoption agency" on elements that aren't in scope. · 03f163c7

Andrew Balholm authored Oct 28, 2011

Pass tests1.dat, test 55:
<!DOCTYPE html><font><table></font></table></font>

| <!DOCTYPE html>
| <html>
|   <head>
|   <body>
|     <font>
|       <table>

Also pass tests through test 69:
<DIV> abc <B> def <I> ghi <P> jkl

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

03f163c7

fmt: handle os.Error values · 0e81e508

Russ Cox authored Oct 27, 2011

Handling os.Error is no different than handling fmt.Stringer
here, so the code is redundant now, but it will be necessary
once error goes in.

Adding it now will make gofix fix it.

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

0e81e508

template: do not use error as stringer · 853c8463
Russ Cox authored Oct 27, 2011
```
R=r
CC=golang-dev
https://golang.org/cl/5305069
```
853c8463

gob: split uses of gobError, remove unnecessary embedding · c0a0fd6c

Russ Cox authored Oct 27, 2011

Will make gofix for error run more smoothly.
The overloading of gobError appears to be unintentional.

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

c0a0fd6c

strconv: use better errors than os.EINVAL, os.ERANGE · c1178aae
Russ Cox authored Oct 27, 2011
```
R=golang-dev, adg
CC=golang-dev
https://golang.org/cl/5327052
```
c1178aae

crypto/tls: add Error method to alert · 01e9a227

Russ Cox authored Oct 27, 2011

alerts get used as both values and errors.
Rather than introduce an alertError wrapper,
this CL just adds an Error method, which will
satisfy the error interface when the time comes.

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

01e9a227

test: error-related fixes · 64f78c91
Russ Cox authored Oct 27, 2011
```
R=golang-dev, bradfitz
CC=golang-dev
https://golang.org/cl/5328051
```
64f78c91
xml: simplify test · 29fb5d3e
Russ Cox authored Oct 27, 2011
```
R=golang-dev, adg
CC=golang-dev
https://golang.org/cl/5320051
```
29fb5d3e
go/build: avoid os.Error in tests · 134d6e4f
Russ Cox authored Oct 27, 2011
```
R=golang-dev, adg
CC=golang-dev
https://golang.org/cl/5308061
```
134d6e4f

go/doc: remove os.NewError anti-heuristic · 2b0c49f2

Russ Cox authored Oct 27, 2011

It will be obsolete when error happens.

Submitting this now will make the error transition earlier,
at the cost of making a locally-built godoc viewing
/pkg/syscall or /pkg/os have some functions appear
under the Error type as constructors.

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

2b0c49f2

encoding/binary: fix type in test · 7f91a39d

Russ Cox authored Oct 27, 2011

Was working only accidentally.

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

7f91a39d

crypto/openpgp/error: use Error in names of error impl types · 67155517

Russ Cox authored Oct 27, 2011

Will make gofix for error work better.
There is no other indication in this file that
these are actually error implementations.

(They are only used elsewhere.)

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

67155517

doc: A Tour of Go in Japanese · 8a7b2b2f
Andrew Gerrand authored Oct 28, 2011
```
R=golang-dev, dsymonds
CC=golang-dev
https://golang.org/cl/5318058
```
8a7b2b2f
html: fix print argument in test · 785baa86
Russ Cox authored Oct 27, 2011
```
R=nigeltao
CC=golang-dev
https://golang.org/cl/5302069
```
785baa86

runtime: lock the main goroutine to the main OS thread during init · 6808da01

Russ Cox authored Oct 27, 2011

We only guarantee that the main goroutine runs on the
main OS thread for initialization.  Programs that wish to
preserve that property for main.main can call runtime.LockOSThread.
This is what programs used to do before we unleashed
goroutines during init, so it is both a simple fix and keeps
existing programs working.

R=iant, r, dave, dvyukov
CC=golang-dev
https://golang.org/cl/5309070

6808da01

pkg: remove .String() from some print arguments · 32f3770e

Russ Cox authored Oct 27, 2011

I found these by adding a check to govet, but the check
produces far too many false positives to be useful.
Even so, these few seem worth cleaning up.

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

32f3770e

27 Oct, 2011 4 commits

html: allow whitespace text nodes in <head> · 053549ca

Andrew Balholm authored Oct 28, 2011

Pass tests1.dat, test 50:
<!DOCTYPE html><script> <!-- </script> --> </script> EOF

| <!DOCTYPE html>
| <html>
|   <head>
|     <script>
|       " <!-- "
|     " "
|   <body>
|     "-->  EOF"

Also pass tests through test 54:
<!DOCTYPE html><title>U-test</title><body><div><p>Test<u></p></div></body>

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

053549ca

misc/windows: allow to be used for amd64 · c92a499b
Alex Brainman authored Oct 28, 2011
```
R=jdpoirier
CC=golang-dev
https://golang.org/cl/5305048
```
c92a499b

spec: define that initialization is sequential · b7ef3c9a

Russ Cox authored Oct 27, 2011

This is true of the existing implementations, and I think
it is an important property to guarantee.

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

b7ef3c9a

go/parser: test and fix := scoping bug · fd31d9fd
Russ Cox authored Oct 27, 2011
```
R=iant
CC=golang-dev, gri
https://golang.org/cl/5327048
```
fd31d9fd