- 22 Jul, 2015 13 commits
-
-
Austin Clements authored
ae1ea2aa moved trace-related functions from runtime/pprof to runtime/trace, but missed a doc comment and a code comment. Update these to reflect the move. Change-Id: I6e1e8861e5ede465c08a2e3f80b976145a8b32d8 Reviewed-on: https://go-review.googlesource.com/12525Reviewed-by: Dmitry Vyukov <dvyukov@google.com>
-
Hari haran authored
Change-Id: I8633a005da7d0c34a5f074fbd979c8e0fc1fba37 Reviewed-on: https://go-review.googlesource.com/12505Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
-
Ingo Krabbe authored
It is very useful to see which test commands are executed. This is of global use, but I wrote it for #11654. Change-Id: I9bfc8e55d5bef21f4c49b917f58bc9a44aefcade Reviewed-on: https://go-review.googlesource.com/12510Reviewed-by: Russ Cox <rsc@golang.org>
-
Adam Langley authored
https://go-review.googlesource.com/#/c/2421/ contains an unfortunate slip where IsOnCurve is called on the CurveParams rather than the curve. This doesn't really matter, but it's a pain for people doing tricks with crypto/elliptic and means that 1.5 would be a regression for them without this change. See https://groups.google.com/forum/#!topic/golang-dev/i8OPUTYctOk Change-Id: Ifa5f25f9a95d7484cb53d4883cfd78dc58a0f9a7 Reviewed-on: https://go-review.googlesource.com/12506Reviewed-by: Russ Cox <rsc@golang.org>
-
Russ Cox authored
Fixes #11730. Change-Id: I5bc60779a87dc07899dd70659a830996bf7812ca Reviewed-on: https://go-review.googlesource.com/12527Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
-
Russ Cox authored
Fixes #11331. Change-Id: I19b8172421044c301bc136fc8f7bfdadbf880e25 Reviewed-on: https://go-review.googlesource.com/12450Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org> Reviewed-by: Andrew Gerrand <adg@golang.org>
-
Russ Cox authored
Fixes #9063. Change-Id: I536ef1f0b30c94c1ebf7922d84cb2f701b7d8a1a Reviewed-on: https://go-review.googlesource.com/12526Reviewed-by: Adam Langley <agl@golang.org> Run-TryBot: Adam Langley <agl@golang.org> TryBot-Result: Gobot Gobot <gobot@golang.org>
-
Carlos C authored
Change-Id: I69a2b6a99a53c875162be8a7d86455559cd74504 Reviewed-on: https://go-review.googlesource.com/11371Reviewed-by: Russ Cox <rsc@golang.org>
-
Dmitry Vyukov authored
Move tracing functions from runtime/pprof to the new runtime/trace package. Fixes #9710 Change-Id: I718bcb2ae3e5959d9f72cab5e6708289e5c8ebd5 Reviewed-on: https://go-review.googlesource.com/12511Reviewed-by: Russ Cox <rsc@golang.org>
-
Didier Spezia authored
JSON decoding currently fails for null values bound to any type which does implement the JSON Unmarshaler interface without checking for null values (such as time.Time). It also fails for types implementing the TextUnmarshaler interface. The expected behavior of the JSON decoding engine in such case is to process null by keeping the value unchanged without producing any error. Make sure null values are handled by the decoding engine itself, and never passed to the UnmarshalText or UnmarshalJSON methods. Fixes #9037 Change-Id: I261d85587ba543ef6f1815555b2af9311034d5bb Reviewed-on: https://go-review.googlesource.com/9376Reviewed-by: Russ Cox <rsc@golang.org>
-
Andrew Gerrand authored
This is now superseded by golang.org/x/build/cmd/release. Fixes #8472 Change-Id: I59664d84996a0fbb5c90582a4702714b3b3cf302 Reviewed-on: https://go-review.googlesource.com/12500Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
-
David Symonds authored
Fixes #11810. Change-Id: I8453e53a72e242a69ea34eb393999e7291d4358f Reviewed-on: https://go-review.googlesource.com/12502 Run-TryBot: David Symonds <dsymonds@golang.org> Reviewed-by: Rob Pike <r@golang.org>
-
Rob Pike authored
This is a corner case but it is suggested we call it out. Fixes #11798. Change-Id: I2ddb5b363cd2921666dbf03bbf98107697ca40e5 Reviewed-on: https://go-review.googlesource.com/12460Reviewed-by: Ian Lance Taylor <iant@golang.org> Reviewed-by: Russ Cox <rsc@golang.org>
-
- 21 Jul, 2015 13 commits
-
-
Brad Fitzpatrick authored
dist test should not print (especially to stdout) during test registration. This confuses other tools interacting with dist using dist test --list, etc. Change-Id: Ie4f82c13e49590c23a7a235d90ddbc4f5ed81e0b Reviewed-on: https://go-review.googlesource.com/12487Reviewed-by: Ian Lance Taylor <iant@golang.org>
-
Ian Lance Taylor authored
Change-Id: I3d5e46d376953fbdd21ce8c161214eaed6378d84 Reviewed-on: https://go-review.googlesource.com/12490Reviewed-by: Andrew Gerrand <adg@golang.org>
-
Ian Lance Taylor authored
Fixes #11709. Fixed #11449. Change-Id: If8fdb27d3dc25fb7017226d143a29cbebc1374c5 Reviewed-on: https://go-review.googlesource.com/12483Reviewed-by: David Crawshaw <crawshaw@golang.org> Run-TryBot: Ian Lance Taylor <iant@golang.org> TryBot-Result: Gobot Gobot <gobot@golang.org>
-
Robert Griesemer authored
In lieu of the more invasive https://go-review.googlesource.com/#/c/12373/ . Change-Id: I0221783fcaa8af04520c80cd2993d7d542d2c431 Reviewed-on: https://go-review.googlesource.com/12486Reviewed-by: Alan Donovan <adonovan@google.com> Run-TryBot: Robert Griesemer <gri@golang.org> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
-
Ian Lance Taylor authored
That test will install cmd/pack for linux_386; we don't want to change GOROOT in short mode. Change-Id: I4b00c578a99779a13c558208bfd4115f8f0513fa Reviewed-on: https://go-review.googlesource.com/12481 Run-TryBot: Ian Lance Taylor <iant@golang.org> Reviewed-by: David Crawshaw <crawshaw@golang.org> TryBot-Result: Gobot Gobot <gobot@golang.org>
-
Ingo Krabbe authored
Change-Id: I87fa25214fbf24469148a63f4e1e61e261105c16 Reviewed-on: https://go-review.googlesource.com/12470 TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Ian Lance Taylor <iant@golang.org>
-
Ian Lance Taylor authored
Fixes #10607. Change-Id: Ib294cf65b07103b60f62b86b21ff2d07b25136fe Reviewed-on: https://go-review.googlesource.com/12242Reviewed-by: Russ Cox <rsc@golang.org>
-
Brad Fitzpatrick authored
Change-Id: I2f855b9ad1676b3c4efedd764ce99e21c104a4ec Reviewed-on: https://go-review.googlesource.com/12446Reviewed-by: Rob Pike <r@golang.org>
-
Paul Marks authored
Change-Id: Ie02426b2b726170d858de96fdd8c51bfdf20d7dc Reviewed-on: https://go-review.googlesource.com/12376Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
-
Brad Fitzpatrick authored
Fixes #11752 Change-Id: If4196b4bbba4f1485608bfafa98fa54781177273 Reviewed-on: https://go-review.googlesource.com/12325Reviewed-by: Andrew Gerrand <adg@golang.org> Reviewed-by: Rob Pike <r@golang.org> Reviewed-by: Russ Cox <rsc@golang.org>
-
Brad Fitzpatrick authored
Fixes #11781 Change-Id: Idc46a6a4fb5bf1c4d394eadf2d860d7ef75c8ccf Reviewed-on: https://go-review.googlesource.com/12390Reviewed-by: Rob Pike <r@golang.org>
-
Alex Brainman authored
Fixes #11147 Change-Id: Ib31160946a53f6f9b11daea211ff04d186b51b3f Reviewed-on: https://go-review.googlesource.com/12067Reviewed-by: Ian Lance Taylor <iant@golang.org>
-
Matthew Dempsky authored
Fixes #11762. Change-Id: I0c7797a2dfdbbcf137bb287f854037a4a697c6e4 Reviewed-on: https://go-review.googlesource.com/12443Reviewed-by: Robert Griesemer <gri@golang.org> Run-TryBot: Matthew Dempsky <mdempsky@google.com>
-
- 20 Jul, 2015 10 commits
-
-
Brad Fitzpatrick authored
Change-Id: If1c12b43ef467c3ef2cbbaaba1bda4ea98032c7e Reviewed-on: https://go-review.googlesource.com/12430Reviewed-by: Rob Pike <r@golang.org>
-
Brad Fitzpatrick authored
Change-Id: I805bb7ba40c8ed579c4ca796e408995586d219b3 Reviewed-on: https://go-review.googlesource.com/12434Reviewed-by: Andrew Gerrand <adg@golang.org> Reviewed-by: Rob Pike <r@golang.org>
-
Robert Griesemer authored
Fixes #11800. Change-Id: I5eb178331f4ae80821f188e19c0c6fadeb330278 Reviewed-on: https://go-review.googlesource.com/12433Reviewed-by: Alan Donovan <adonovan@google.com>
-
Ian Lance Taylor authored
The cmd/go tests run too long on a Raspberry Pi. I've cut times as much as I can see without more serious steps like not running tests. Fixes #11779. Change-Id: Ice5da052902decea2e6ac32d0f2ce084c39ea1ab Reviewed-on: https://go-review.googlesource.com/12368 Run-TryBot: Ian Lance Taylor <iant@golang.org> Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org> TryBot-Result: Gobot Gobot <gobot@golang.org>
-
Ian Lance Taylor authored
The errors package is smaller and builds faster. Update #11779. Change-Id: Iaf2d43734ec4b0aa325590eb5d38522306b608cb Reviewed-on: https://go-review.googlesource.com/12366Reviewed-by: David Crawshaw <crawshaw@golang.org> TryBot-Result: Gobot Gobot <gobot@golang.org>
-
Ian Lance Taylor authored
Speed up the test suite by building the errors package rather than the strings package in some cases where the specific package we are building doesn't matter. The errors package is smaller, and doesn't have any assembler code. Also make a couple of tests run in parallel. Update #11779. Change-Id: I62e47f8655f9d85bf93c70ae6e6121276d96aee0 Reviewed-on: https://go-review.googlesource.com/12365 Run-TryBot: Ian Lance Taylor <iant@golang.org> Reviewed-by: David Crawshaw <crawshaw@golang.org> TryBot-Result: Gobot Gobot <gobot@golang.org>
-
Ian Lance Taylor authored
It changes GOROOT, so we shouldn't run it in short mode. Also, it's fairly slow. Update #11779. Change-Id: I3d3344954cf9b2ac70070c878a67cb65ac8fd85c Reviewed-on: https://go-review.googlesource.com/12364 Run-TryBot: Ian Lance Taylor <iant@golang.org> Reviewed-by: Andrew Gerrand <adg@golang.org>
-
Ian Lance Taylor authored
On my laptop reduces time required for test from 22 seconds to 0.14 seconds. Update #11779. Change-Id: I715d85bd9c6f7683c6915eedd2539813aa5efc58 Reviewed-on: https://go-review.googlesource.com/12363Reviewed-by: David Crawshaw <crawshaw@golang.org> Reviewed-by: Andrew Gerrand <adg@golang.org>
-
Michael Hudson-Doyle authored
This is mostly Russ's https://golang.org/cl/12145 but with some extra fixes to account for the fact that function declarations without implementations now break shared libraries, and including my test case. Fixes #11480. Change-Id: Iabdc2934a0378e5025e4e7affadb535eaef2c8f1 Reviewed-on: https://go-review.googlesource.com/12340Reviewed-by: Ian Lance Taylor <iant@golang.org>
-
Brad Fitzpatrick authored
Change-Id: I6a8bb853a538c80d95589321d3226784bc017eef Reviewed-on: https://go-review.googlesource.com/12327Reviewed-by: Andrew Gerrand <adg@golang.org>
-
- 19 Jul, 2015 1 commit
-
-
Austin Clements authored
The runtime.GC documentation was rewritten in df2809f0 to make it clear that it blocks until GC is complete, but the re-rewrite in ed9a4c91 and e28a6792 lost this property when clarifying that it may also block the entire program and not just the caller. Try to arrive at wording that conveys both of these properties. Change-Id: I1e255322aa28a21a548556ecf2a44d8d8ac524ef Reviewed-on: https://go-review.googlesource.com/12392Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org> Reviewed-by: Rob Pike <r@golang.org>
-
- 18 Jul, 2015 3 commits
-
-
Ian Lance Taylor authored
The findmoduledatap function will not return nil in ordinary use, but check for nil to try to avoid crashing when we are already crashing. Update #11783. Change-Id: If7b1adb51efab13b4c1a37b6f3c9ad22641a0b56 Reviewed-on: https://go-review.googlesource.com/12391 Run-TryBot: Ian Lance Taylor <iant@golang.org> Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
-
Dmitry Vyukov authored
Fixes #11766 Fixes #11769 Fixes #11767 Fixes #11770 Change-Id: I441382af58f60deb46dcdd70076763b2c47738d4 Reviewed-on: https://go-review.googlesource.com/12378Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org> Run-TryBot: Dmitry Vyukov <dvyukov@google.com> TryBot-Result: Gobot Gobot <gobot@golang.org>
-
Silvan Jegen authored
The default value for error is nil so there is no need to assign this value here. Change-Id: I4714ef7607996ccbf91b704390e1d1d39ee3847b Reviewed-on: https://go-review.googlesource.com/12355Reviewed-by: Ian Lance Taylor <iant@golang.org> Run-TryBot: Ian Lance Taylor <iant@golang.org> TryBot-Result: Gobot Gobot <gobot@golang.org>
-