Commits · 7f91a39d3d520d99f988d7060237550f11b6ab18 · Kirill Smelkov / go

28 Oct, 2011 6 commits

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 6 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

goinstall: More intelligent vcs selection for common sites · d066e02a

Julian Phillips authored Oct 27, 2011

goinstall has built in support for a few common code hosting sites.  The
identification of which vcs tool should be used was based purely on a
regex match against the provided import path.  The problem with this
approach is that it requires distinct import paths for different vcs
tools on the same site.

Since bitbucket has recently starting hosting Git repositories under the
same bitbucket.org/user/project scheme as it already hosts Mercurial
repositories, now would seem a good time to take a more flexible
approach.

We still match the import path against a list of regexes, but now the
match is purely to distinguish the different hosting sites.  Once the
site is identified, the specified function is called with the repo and
path matched out of the import string.  This function is responsible for
creating the vcsMatch structure that tells us what we need to download
the code.

For github and launchpad, only one vcs tool is currently supported, so
these functions can simply return a vcsMatch structure.  For googlecode,
we retain the behaviour of determing the vcs from the import path - but
now it is done by the function instead of the regex.  For bitbucket, we
use api.bitbucket.org to find out what sort of repository the specified
import path corresponds to - and then construct the appropriate vcsMatch
structure.

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

d066e02a

doc: fix escaping in weekly.html · 1e900b0d
Andrew Gerrand authored Oct 27, 2011
```
R=golang-dev, dsymonds
CC=golang-dev
https://golang.org/cl/5306070
```
1e900b0d

26 Oct, 2011 28 commits

html: parse <style> elements inside <head> element. · 833fb419

Andrew Balholm authored Oct 27, 2011

Also correctly handle EOF inside a <style> element.

Pass tests1.dat, test 49:
<!DOCTYPE html><style> EOF

| <!DOCTYPE html>
| <html>
|   <head>
|     <style>
|       " EOF"
|   <body>

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

833fb419

html: close <option> element when opening <optgroup> · bd07e4f2

Andrew Balholm authored Oct 27, 2011

Pass tests1.dat, test 34:
<!DOCTYPE html>A<option>B<optgroup>C<select>D</option>E

| <!DOCTYPE html>
| <html>
|   <head>
|   <body>
|     "A"
|     <option>
|       "B"
|     <optgroup>
|       "C"
|       <select>
|         "DE"

Also passes tests 35-48. Test 48 is:
</ COM--MENT >

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

bd07e4f2

6l: remove mention of -e flag · 2cf53598

Russ Cox authored Oct 26, 2011

Emitting ELF symbol table is the default behavior now
and cannot be disabled.

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

2cf53598

cc: change cas to newcase · 1bc1caa8

Ron Minnich authored Oct 26, 2011

Change the name of cas() in cc to newcase() to avoid a NIX conflict.
cas() is used in cc to create a new Case struct. There is a name
conflict in that cas() is a commonly-used
name for compare and swap. Since cas() is only used internally
in the compiler in 3 places, change the name to avoid a wider
conflict with the NIX runtime. This issue might well come up on
other OSes in the future anyway, as the name is fairly common.

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

1bc1caa8

test/alias.go: additional tests · 8658b36b
Russ Cox authored Oct 26, 2011
```
R=ken2
CC=golang-dev
https://golang.org/cl/5327045
```
8658b36b

misc/vim: Add rune keyword · 0d8fb375

Jongmin Kim authored Oct 27, 2011

According to adding rune type

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

0d8fb375

gofix: test import insertion, deletion · 6323a40f

Russ Cox authored Oct 26, 2011

Small change to go/ast, go/parser, go/printer so that
gofix can delete the blank line left from deleting an import.

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

6323a40f

A+C: Ron Minnich (individual CLA) · 8fee9bc8
Russ Cox authored Oct 26, 2011
```
R=golang-dev, adg
CC=golang-dev
https://golang.org/cl/5308057
```
8fee9bc8
A+C: Jan Newmarch (individual CLA) · bff71ed4
Russ Cox authored Oct 26, 2011
```
R=golang-dev, iant
CC=golang-dev
https://golang.org/cl/5305062
```
bff71ed4
C+A: add Jongmin Kim (individual CLA) · 9611d12b
Brad Fitzpatrick authored Oct 26, 2011
```
R=golang-dev, rsc, iant
CC=golang-dev
https://golang.org/cl/5312057
```
9611d12b

crypto/rsa: change public exponent from 3 to 65537 · 4403e6b6

Adam Langley authored Oct 26, 2011

Although there's still no concrete security reason not to use 3, I
think Bleichenbacher has convinced me that it's a useful defense and
it's what everyone else does.

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

4403e6b6

net: document why we do not use SO_REUSEADDR on windows · c3733b29
Alex Brainman authored Oct 26, 2011
```
R=rsc, adg
CC=golang-dev
https://golang.org/cl/5302058
```
c3733b29
.hgignore: ignore src/pkg/exp/ebnflint/ebnflint · 704bf779
Alex Brainman authored Oct 26, 2011
```
R=golang-dev, adg
CC=golang-dev
https://golang.org/cl/5294060
```
704bf779
gc: fix [568]g -V crash · 8b92066e
Mikio Hara authored Oct 26, 2011
```
R=lvd
CC=golang-dev
https://golang.org/cl/5314060
```
8b92066e

redo CL 5302057 / dac58d9c9e4a · f777be8f

Andrew Gerrand authored Oct 26, 2011

««« original CL description
http: remove Connection header in ReverseProxy

Fixes #2342

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

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

f777be8f

tag weekly.2011-10-26 · d12f66a8
Andrew Gerrand authored Oct 26, 2011
```
R=rsc
CC=golang-dev
https://golang.org/cl/5304063
```
d12f66a8
weekly.2011-10-26 (new rune type) · 659f1f20
Andrew Gerrand authored Oct 26, 2011
```
R=rsc
CC=golang-dev
https://golang.org/cl/5297062
```
659f1f20

exp/norm: use rune · c945f77f