Commits · cfed26c7ceeaf093d975e0a748d270cd1b716816 · Kirill Smelkov / go

07 Aug, 2014 5 commits
- misc/nacl: wrap lines in README file · cfed26c7
  Andrew Gerrand authored Aug 07, 2014
```
LGTM=dan.kortschak, dave
R=dave, dan.kortschak
CC=golang-codereviews
https://golang.org/cl/121350043
```
  cfed26c7
- C: add Paul Nasrat (Google CLA) · 9f07456c
  Andrew Gerrand authored Aug 07, 2014
```
TBR=gobot
R=golang-codereviews
CC=golang-codereviews
https://golang.org/cl/125790043
```
  9f07456c
- doc/go1.4.txt: add support for ALPN · cdf77676
  Mikio Hara authored Aug 07, 2014
```
LGTM=minux
R=r, agl, minux
CC=golang-codereviews
https://golang.org/cl/121340043
```
  cdf77676
- doc/go1.4.txt: implement monotonic clocks on windows · 7ce3406c
  Alex Brainman authored Aug 07, 2014
```
LGTM=dave
R=golang-codereviews, dave
CC=golang-codereviews
https://golang.org/cl/126760043
```
  7ce3406c
- path/filepath: do not restore original working directory twice in test · a2d3669e
  Alex Brainman authored Aug 07, 2014
```
LGTM=dave
R=golang-codereviews, dave
CC=golang-codereviews
https://golang.org/cl/122910043
```
  a2d3669e
06 Aug, 2014 35 commits

encoding/xml: add InputOffset method to Decoder · b91aea55

Russ Cox authored Aug 06, 2014

Among other things, this allows users to match the decoded
pieces with the original XML, which can be necessary for
implementing standards like XML signatures.

Fixes #8484.

LGTM=bradfitz
R=bradfitz
CC=golang-codereviews
https://golang.org/cl/122960043

b91aea55

cmd/go: revise disallowInternal · fef54b22

Russ Cox authored Aug 06, 2014

This fixes two problems: x/internal/y/z was using parent = x/internal/y instead of x,
and hasPathPrefix only looks at /, not \ for Windows.

LGTM=bradfitz
R=bradfitz
CC=golang-codereviews
https://golang.org/cl/121280045

fef54b22

debug/dwarf: fix typos in comment for UnspecifiedType · 7bcaff70
Rob Pike authored Aug 06, 2014
```
LGTM=iant, bradfitz
R=bradfitz, iant
CC=golang-codereviews
https://golang.org/cl/120700043
```
7bcaff70

runtime: clean up naming of mcallable functions. · e359bea8

Keith Randall authored Aug 06, 2014

Introduce the mFunction type to represent an mcall/onM-able function.
Name such functions using _m.

LGTM=bradfitz
R=bradfitz
CC=golang-codereviews
https://golang.org/cl/121320043

e359bea8

test/mapnan.go: add regression test for non-empty interfaces. · 161ba662
Alan Donovan authored Aug 06, 2014
```
LGTM=rsc, khr
R=rsc, khr, bradfitz
CC=golang-codereviews
https://golang.org/cl/126720043
```
161ba662

runtime: use better hash for floating point inputs · 5fbcdb26

Russ Cox authored Aug 06, 2014

Hashing on the bytes instead of the words does
a (much) better job of using all the bits, so that
maps of floats have linear performance.

LGTM=khr
R=golang-codereviews, khr
CC=adonovan, golang-codereviews
https://golang.org/cl/126720044

5fbcdb26

cmd/go: implement 'internal' convention · 1338f327

Russ Cox authored Aug 06, 2014

See golang.org/s/go14internal for design.

LGTM=r
R=r, adg
CC=golang-codereviews
https://golang.org/cl/120600043

1338f327

runtime: shorten hash declarations · ea3ac6ba
Keith Randall authored Aug 06, 2014
```
LGTM=iant
R=dvyukov, iant
CC=golang-codereviews
https://golang.org/cl/117680044
```
ea3ac6ba

runtime: use better hash for non-empty interface · f098a296

Russ Cox authored Aug 06, 2014

The implementation 'return 0' results in too many collisions.

LGTM=khr
R=golang-codereviews, adonovan, khr
CC=golang-codereviews, iant, khr, r
https://golang.org/cl/125720044

f098a296

codereview: do not hit upload_complete on first patch · 2185dbcd

Russ Cox authored Aug 06, 2014

Causes server 500 error, so don't do it.

TBR=minux
CC=golang-codereviews
https://golang.org/cl/125770043

2185dbcd

codereview: fix submit of merge + include branch prefix in submitted message · 4cc8be60

Russ Cox authored Aug 06, 2014

hg insists that we not list explicit files for a merge.

for benefit of other tools reading logs, include branch prefix
at start of every commit message.

LGTM=minux
R=minux
CC=golang-codereviews
https://golang.org/cl/124780044

4cc8be60

cmd/gc: make liveness ~10x faster · 7aa3031e

Russ Cox authored Aug 06, 2014

1) The arrayindexof lookup function is O(n). Replace with O(1) lookups.

2) The checkptxt function is O(n²) and is purely for debugging.
Only run when the debugging flags are turned on.

3) Iterating over sparse bitmaps can be done faster word by word.
Introduce and use bvnext for that.

Run times before and after, on my 2.5 GHz Core i5 MacBook Pro.

x.go       9.48  0.84  issue 8259

x100.go    0.01  0.01  issue 8354
x1000.go   0.10  0.10
x2000.go   0.62  0.19
x3000.go   1.33  0.34
x4000.go   2.29  0.49
x5000.go   3.89  0.67
x6000.go   5.00  0.90
x7000.go   6.70  1.13
x8000.go   9.44  1.38
x9000.go  11.23  1.87
x10000.go 13.78  2.09

Fixes #8259.
Fixes #8354.

LGTM=iant, r
R=golang-codereviews, iant, r
CC=golang-codereviews
https://golang.org/cl/125720043

7aa3031e

codereview: handle upload of merge · c1fcdb0e
Russ Cox authored Aug 06, 2014
```
LGTM=minux
R=minux
CC=golang-codereviews
https://golang.org/cl/118690043
```
c1fcdb0e
codereview: preserve branch prefix in subject during hg mail · 6ff5c317
Russ Cox authored Aug 06, 2014
```
LGTM=minux
R=minux
CC=golang-codereviews
https://golang.org/cl/124800043
```
6ff5c317

debug/dwarf: fix Reader panic on DW_TAG_unspecified_type · c0ccf6f3

Derek Parker authored Aug 06, 2014

The linker currently produces the DWARF 3 DW_TAG_unspecified_type tag, however the Reader in debug/dwarf will panic whenever that tag is encountered.

Fixes #8437.

LGTM=rsc
R=golang-codereviews, bradfitz, iant, rsc
CC=golang-codereviews
https://golang.org/cl/117280043

c0ccf6f3

A+C: Derek Parker (individual CLA) · 059f93be
Brad Fitzpatrick authored Aug 06, 2014
```
Generated by a+c.

R=gobot
CC=golang-codereviews
https://golang.org/cl/125750043
```
059f93be

codereview: enable work and code reviews in development branches · 69c2e2b9

Russ Cox authored Aug 06, 2014

This is an experiment. See mail on golang-dev
(subject: "an experiment: development branches").

LGTM=minux
R=minux
CC=golang-codereviews
https://golang.org/cl/117660043

69c2e2b9

runtime: turn off 'unexpected return pc' print on arm traceback · e5e547c7

Russ Cox authored Aug 06, 2014

It can happen legitimately if a profiling signal arrives at just the wrong moment.
It's harmless.

Fixes #8153.

LGTM=minux
R=golang-codereviews, minux
CC=golang-codereviews, iant, r
https://golang.org/cl/118670043

e5e547c7

crypto/tls: Added dynamic alternative to NameToCertificate map for SNI · 14e8885c

Percy Wegmann authored Aug 06, 2014

Revised version of https://golang.org/cl/81260045/

LGTM=agl
R=golang-codereviews, gobot, agl, ox
CC=golang-codereviews
https://golang.org/cl/107400043

14e8885c

cmd/cgo: consistently map void* to *byte under -{c,go}defs · 2b9f3fce
Matthew Dempsky authored Aug 06, 2014
```
Fixes #8478.

LGTM=iant
R=iant
CC=golang-codereviews
https://golang.org/cl/122150043
```
2b9f3fce

runtime: remove unused variable · 7dfcebbd

Dmitriy Vyukov authored Aug 06, 2014

Left over from cl/119490044.

LGTM=bradfitz
R=rsc, bradfitz
CC=golang-codereviews
https://golang.org/cl/125730043

7dfcebbd

runtime: simplify code · f3dd6df6

Dmitriy Vyukov authored Aug 06, 2014

Full spans can't be passed to UncacheSpan since we get rid of free.

LGTM=rsc
R=golang-codereviews
CC=golang-codereviews, khr, rsc
https://golang.org/cl/119490044

f3dd6df6

dist: fix Plan 9 build · de1ab89f

David du Colombier authored Aug 06, 2014

Since CL 115060044, mkanames declares an empty
array in anames8.c and anames6.c, which is not
valid for the Plan 9 compiler.

char* cnames8[] = {
};

This change makes mkanames not declaring the
cnames array when no C_ constants are found.

LGTM=iant
R=minux, iant
CC=golang-codereviews
https://golang.org/cl/117680043

de1ab89f

runtime: burn cpu before calling yield in windows runtime.systime · bf13b711
Alex Brainman authored Aug 06, 2014
```
LGTM=dvyukov
R=golang-codereviews, dvyukov
CC=golang-codereviews
https://golang.org/cl/117670043
```
bf13b711

undo CL 114420043 / b613f2acdf69 · 6503d400

Shenghou Ma authored Aug 06, 2014

Broke freebsd/amd64 due to exposure of a latent bug.

««« original CL description
cmd/ld: fix operator precedence

LGTM=rsc
R=rsc, iant
CC=golang-codereviews
https://golang.org/cl/114420043
»»»

TBR=dfc
R=dave
CC=golang-codereviews
https://golang.org/cl/120630043

6503d400

liblink: encode MOVBQZX as MOVZBL instead of MOVZBQ · bbc6bb9a
Rui Ueyama authored Aug 05, 2014
```
LGTM=rsc
R=golang-codereviews, gobot, rsc
CC=golang-codereviews
https://golang.org/cl/118480046
```
bbc6bb9a

liblink, cmd/dist, cmd/5l: introduce %^ and move C_* constants. · d69d0fe9

Shenghou Ma authored Aug 06, 2014

The helps certain diagnostics and also removed duplicated enums as a side effect.

LGTM=dave, rsc
R=rsc, dave
CC=golang-codereviews
https://golang.org/cl/115060044

d69d0fe9

cmd/cc, cmd/gc: set ctxt->debugvlog with debug['v'] · ae027f52
Shenghou Ma authored Aug 06, 2014
```
LGTM=rsc
R=rsc, iant
CC=golang-codereviews
https://golang.org/cl/118610043
```
ae027f52
cmd/5l, cmd/6l, cmd/8l, cmd/ld: remove unused code, consolidate enums · 45bd2f8c
Shenghou Ma authored Aug 06, 2014
```
LGTM=rsc
R=rsc, iant
CC=golang-codereviews
https://golang.org/cl/120220043
```
45bd2f8c
liblink: support big-endian properly · 08ee2661
Shenghou Ma authored Aug 06, 2014
```
LGTM=rsc
R=rsc, iant
CC=golang-codereviews
https://golang.org/cl/115300044
```
08ee2661
cmd/ld: fix operator precedence · 2c181f03
Shenghou Ma authored Aug 06, 2014
```
LGTM=rsc
R=rsc, iant
CC=golang-codereviews
https://golang.org/cl/114420043
```
2c181f03

runtime: get rid of SA_RESTORER on ARM. · de14137b

Shenghou Ma authored Aug 06, 2014

The manpages says SA_RESTORER is obsolete, and indeed, not every architecture
support it. However, sadly it's required on x86_64, see http://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/tree/arch/x86/kernel/signal.c?id=26bcd8b72563b4c54892c4c2a409f6656fb8ae8b#n430, so only use it on x86.

LGTM=rsc
R=rsc, iant
CC=golang-codereviews
https://golang.org/cl/115450043

de14137b

liblink: use LinkArch.textflag() to get text and dataflag · 3c24b60f

Shenghou Ma authored Aug 06, 2014

Rather than switch on thechar.

LGTM=rsc, dave
R=rsc, iant, dave
CC=golang-codereviews
https://golang.org/cl/119330043

3c24b60f

runtime: remove dead code · 61b1d3a7

Shenghou Ma authored Aug 06, 2014

LGTM=bradfitz, dave, ruiu
R=rsc, iant, bradfitz, dave, ruiu
CC=golang-codereviews
https://golang.org/cl/116610043

61b1d3a7

runtime: disable aeshash on NaCl at compile time · 18ab35f7

Shenghou Ma authored Aug 06, 2014

Fixes build for nacl/386.

LGTM=dave
R=khr, bradfitz, dave, dan.kortschak, rsc
CC=golang-codereviews
https://golang.org/cl/121080043

18ab35f7