- 22 Dec, 2013 1 commit
-
-
Ian Lance Taylor authored
Generated by addca. R=gobot CC=golang-codereviews https://golang.org/cl/42800045
-
- 21 Dec, 2013 4 commits
-
-
Shawn Smith authored
bytes, strings: improve Title test coverage by adding cases with underscore and unicode line separator R=golang-codereviews, gobot, r CC=golang-codereviews https://golang.org/cl/42310045
-
Alex Brainman authored
R=golang-codereviews, r CC=golang-codereviews https://golang.org/cl/44160044
-
David du Colombier authored
R=golang-codereviews, rsc CC=golang-codereviews, jas https://golang.org/cl/44790043
-
Russ Cox authored
Reduces 6g big.go memory usage from 251 MB to 242 MB. Reduces 6g slow.go memory usage from 529 MB to 453 MB. Mostly a memory savings; 6g slow.go is only about 5% faster. The test programs are at https://rsc.googlecode.com/hg/testdata/big.go (36k lines, 276kB) https://rsc.googlecode.com/hg/testdata/slow.go (7k lines, 352kB) R=golang-codereviews, bradfitz, iant CC=golang-codereviews https://golang.org/cl/42280045
-
- 20 Dec, 2013 13 commits
-
-
Jakub Ryszard Czarnowicz authored
RFC 2616, section 7.2.1 - empty type SHOULD be treated as application/octet-stream. Fixes #6616. R=golang-codereviews, gobot, bradfitz, josharian CC=golang-codereviews https://golang.org/cl/31810043
-
Russ Cox authored
As much as 7x speedup on some programs, cuts all.bash time by 20%. Change splicebefore function from O(n) to O(1). The approach was suggested by Carl during the code's review but apparently did not make it into the tree. It makes a huge difference on huge programs. Make twobitwalktype1 slightly faster by using & instead of %. Really it needs to be cached; left a note to that effect. (Not a complete fix, hence the ½.) big.go (output of test/chan/select5.go) 47.53u 0.50s 48.14r before this CL 7.09u 0.47s 7.59r with splicebefore change (6.7x speedup) 6.15u 0.42s 6.59r with twobitwalktype1 change (1.15x speedup; total 7.7x) slow.go (variant of program in go.text, by mpvl) 77.75u 2.11s 80.03r before this CL 24.40u 1.97s 26.44r with splicebefore change (3.2x speedup) 18.12u 2.19s 20.38r with twobitwalktype1 change (1.35x speedup; total 4.3x) test/run 150.63u 49.57s 81.08r before this CL 88.01u 45.60s 46.65r after this CL (1.7x speedup) all.bash 369.70u 115.64s 256.21r before this CL 298.52u 110.35s 214.67r after this CL (1.24x speedup) The test programs are at https://rsc.googlecode.com/hg/testdata/big.go (36k lines, 276kB) https://rsc.googlecode.com/hg/testdata/slow.go (7k lines, 352kB) R=golang-codereviews, gobot, r CC=cshapiro, golang-codereviews https://golang.org/cl/43210045
-
Russ Cox authored
Eventually we will want to bypass DATA for everything, but the relocations are not standardized well enough across architectures to make that possible. This did not help as much as I expected, but it is definitely better. It shaves maybe 1-2% off all.bash depending on how much you trust the timings of a single run: Before: 241.139r 362.702u 112.967s After: 234.339r 359.623u 111.045s R=golang-codereviews, gobot, r, iant CC=golang-codereviews https://golang.org/cl/44650043
-
Rob Pike authored
Expand the type's doc comment to make its purpose clear and discourage misuse. R=golang-codereviews, gobot, rsc CC=golang-codereviews https://golang.org/cl/44680043
-
Brad Fitzpatrick authored
Generated by addca. R=gobot CC=golang-codereviews https://golang.org/cl/44690043
-
Josh Bleecher Snyder authored
R=adg, bradfitz CC=golang-codereviews https://golang.org/cl/44430043
-
Brad Fitzpatrick authored
And merge the blackhole.go file back into ioutil, where it once was. It was only in a separate file because it used to have race-vs-!race versions. R=golang-codereviews, rsc CC=golang-codereviews https://golang.org/cl/44060044
-
Russ Cox authored
These no longer work; removing them makes other refactoring easier. The code for pack P being deleted in this CL does not work either. I created issue 6989 to track restoring this functionality (probably not until pack is written in Go). R=golang-codereviews, bradfitz CC=golang-codereviews https://golang.org/cl/44300043
-
Adam Langley authored
The practice of storing reference connections for testing has worked reasonably well, but the large blocks of literal data in the .go files is ugly and updating the tests is a real problem because their number has grown. This CL changes the way that reference tests work. It's now possible to automatically update the tests and the test data is now stored in testdata/. This should make it easier to implement changes that affect all connections, like implementing the renegotiation extension. R=golang-codereviews, r CC=golang-codereviews https://golang.org/cl/42060044
-
Russ Cox authored
See this thread for background: https://groups.google.com/forum/#!topic/golang-dev/xG7vPi21r8g R=golang-codereviews, gobot, golang-dev, dsymonds CC=golang-codereviews, golang-dev https://golang.org/cl/44450043
-
Alex A Skinner authored
Per RFC 4291, 'The use of "::" indicates one or more groups of 16 bits of zeros.' Fixes #6628 R=golang-dev, rsc, minux.ma, mikioh.mikioh CC=golang-dev https://golang.org/cl/15990043
-
Shenghou Ma authored
R=golang-dev CC=golang-dev https://golang.org/cl/38030045
-
Shenghou Ma authored
Fixes #6988. R=golang-dev, iant CC=golang-dev https://golang.org/cl/44070046
-
- 19 Dec, 2013 18 commits
-
-
Keith Randall authored
Needed for precise gc and copying stacks. reflect.Value now takes 4 words instead of 3. Still to do: - un-iword-ify channel ops. - un-iword-ify method receivers. R=golang-dev, iant, rsc, khr CC=golang-dev https://golang.org/cl/43040043
-
Russ Cox authored
This makes the linker's -X flag work again. R=iant CC=bradfitz, golang-codereviews, golang-dev https://golang.org/cl/44360043
-
Brad Fitzpatrick authored
Notably, to show allocs. Currently: 11766 B/op, 21 allocs/op, at least one alloc of which is in the benchmark loop itself. R=golang-dev, jnewlin CC=golang-dev https://golang.org/cl/40370057
-
Rémy Oudompheng authored
The runtime tests are executed 4 times in all.bash and there is currently a 5-second delay each time. R=golang-dev, minux.ma, khr, bradfitz CC=golang-dev https://golang.org/cl/42450043
-
David du Colombier authored
R=golang-dev, lucio.dere, rsc CC=golang-dev https://golang.org/cl/44080046
-
Rob Pike authored
The code is all about tags, and the cmd/go documentation said to look in the go/build documentation for information about tags, but the documentation said nothing about tags, only build constraints. Make things clearer. R=golang-dev, adg, rsc CC=golang-dev https://golang.org/cl/44100043
-
Adam Langley authored
Since SHA-256 is now the default hash function, x509 should import it otherwise some programs may fail because it hasn't been linked in. R=golang-dev, dave, minux.ma CC=golang-dev https://golang.org/cl/44010047
-
ChaiShushan authored
1. expr append executable extension. 2. support '\r' character. Fixes #6851. R=golang-dev, rsc CC=golang-dev https://golang.org/cl/35330043
-
Joel Sing authored
Make hostobj work on OpenBSD 5.3/5.4/-current - these have PIE enabled by default and linking fails since the Go linker generates objects that are neither PIC nor PIE. Fixes #5067 R=golang-dev, rsc CC=golang-dev https://golang.org/cl/7572049
-
Mikio Hara authored
To make it possible to type "go test -run=DNS". Also removes unnecessary parens. R=golang-dev, bradfitz CC=golang-dev https://golang.org/cl/43470051
-
Alex Brainman authored
Fixes #6338 R=golang-dev, kin.wilson.za, rsc CC=golang-dev https://golang.org/cl/36180044
-
Alex Brainman authored
R=golang-dev, bradfitz CC=golang-dev https://golang.org/cl/44180043
-
Brad Fitzpatrick authored
We did this elsewhere when io.ByteReader was added but forgot this one. R=golang-dev, khr CC=golang-dev https://golang.org/cl/43480052
-
Keith Randall authored
On the plus side, we don't need to change the bits when mallocing pointerless objects. On the other hand, we need to mark objects in the free lists during GC. But the free lists are small at GC time, so it should be a net win. benchmark old ns/op new ns/op delta BenchmarkMalloc8 40 33 -17.65% BenchmarkMalloc16 45 38 -15.72% BenchmarkMallocTypeInfo8 58 59 +0.85% BenchmarkMallocTypeInfo16 63 64 +1.10% R=golang-dev, rsc, dvyukov CC=cshapiro, golang-dev https://golang.org/cl/41040043
-
Mikio Hara authored
Also removes an unused variable. Fixes #6795. R=adg, dave, bradfitz, gobot CC=golang-dev https://golang.org/cl/29440043
-
Brad Fitzpatrick authored
For machines, not threads. Update #4720 R=golang-dev, rsc CC=golang-dev https://golang.org/cl/44150043
-
Russ Cox authored
Overloading size leads to problems if clients try to set up an LSym by hand. R=golang-dev, bradfitz CC=golang-dev https://golang.org/cl/44140043
-
Russ Cox authored
R=r, bradfitz CC=golang-dev https://golang.org/cl/43480049
-
- 18 Dec, 2013 4 commits
-
-
David du Colombier authored
R=golang-dev, rsc CC=golang-dev https://golang.org/cl/43480050
-
Brad Fitzpatrick authored
Update #4720 R=golang-dev, iant CC=golang-dev https://golang.org/cl/44080043
-
Brad Fitzpatrick authored
Benchmark is within the noise. I had to run this a dozen times each before & after (on wall power, without a browser running) before I could get halfway consistent numbers, and even then they jumped all over the place, with the new one sometimes being better. But these are the best of a dozen each. Slowdown is expected anyway, since I imagine channels are optimized more. benchmark old ns/op new ns/op delta BenchmarkCodeEncoder 26556987 27291072 +2.76% BenchmarkEncoderEncode 1069 1071 +0.19% benchmark old MB/s new MB/s speedup BenchmarkCodeEncoder 73.07 71.10 0.97x benchmark old allocs new allocs delta BenchmarkEncoderEncode 2 2 0.00% benchmark old bytes new bytes delta BenchmarkEncoderEncode 221 221 0.00% Update #4720 R=golang-dev, iant CC=golang-dev https://golang.org/cl/37720047
-
Dave Cheney authored
Address several warnings generated by clang -fsanitize=undefined R=golang-dev, iant CC=golang-dev https://golang.org/cl/43050043
-