- 26 Jul, 2015 1 commit
-
-
Jeff R. Allen authored
Update the docs to explain the code added in commit 67e1d400. Fixes #11831. Change-Id: I8fe72e449507847c4bd9d77de40947ded7f2ff9d Reviewed-on: https://go-review.googlesource.com/12515Reviewed-by:
Dave Cheney <dave@cheney.net>
-
- 24 Jul, 2015 4 commits
-
-
Matthew Dempsky authored
Change-Id: I102901e3df76830ccd5ab74d757203d103eef9e8 Reviewed-on: https://go-review.googlesource.com/12657Reviewed-by:
Brad Fitzpatrick <bradfitz@golang.org>
-
Ian Lance Taylor authored
Issue 11214 reports problems with older versions of gdb. It does work with gdb 7.9 on my Ubuntu Trusty system, so take that as the minimum required version. Fixes #11214. Change-Id: I61b732895506575be7af595f81fc1bcf696f58c2 Reviewed-on: https://go-review.googlesource.com/12626Reviewed-by:
Austin Clements <austin@google.com>
-
Ian Lance Taylor authored
This adds documentation for all the environment variables I could locate in the go tool and the commands that it invokes. Fixes #9672. Change-Id: Id5f09160a3a8a938af4a3fcb8757eb3eced05416 Reviewed-on: https://go-review.googlesource.com/12620Reviewed-by:
Rob Pike <r@golang.org>
-
Ian Lance Taylor authored
Fixes #11847. Change-Id: I21736a4c6f6fb2f61aec1396ce2c965e3e329e92 Reviewed-on: https://go-review.googlesource.com/12621Reviewed-by:
Mikio Hara <mikioh.mikioh@gmail.com>
-
- 23 Jul, 2015 16 commits
-
-
Damian Gryski authored
In https://go-review.googlesource.com/#/c/8611/ , these tests were supposed to be skipped only for linux and darwin, as the comment says. This patch fixes the logic in the if test. Change-Id: Iff0a32186267457a414912c4c3ee4495650891a2 Reviewed-on: https://go-review.googlesource.com/12517Reviewed-by:
Josh Bleecher Snyder <josharian@gmail.com> Reviewed-by:
Robert Griesemer <gri@golang.org> Run-TryBot: Robert Griesemer <gri@golang.org>
-
David Chase authored
Old code appended, did not play well with a closure with a ... param. Fixes #11075. Change-Id: Ib7c8590c5c4e576e798837e7499e00f3494efb4a Reviewed-on: https://go-review.googlesource.com/12580Reviewed-by:
Russ Cox <rsc@golang.org> Run-TryBot: David Chase <drchase@google.com>
-
Alan Donovan authored
This reverts commit b1177d39. Change-Id: Iffbd63d5993633143a81572b223e3c5dd2353206 Reviewed-on: https://go-review.googlesource.com/12581Reviewed-by:
Alan Donovan <adonovan@google.com>
-
Ian Lance Taylor authored
On Windows, gcc -o foo will generate foo.exe. Prevent that from happening by adding a final '.' if necessary so that GCC thinks that the file already has an extension. Also remove the initial output file when doing an external link, and use mayberemoveoutfile, not os.Remove, when building an archive (otherwise we will do the wrong thing for -buildmode=c-archive -o /dev/null). I didn't add a test, as it requires using cgo and -o on Windows. Fixes #11725. Change-Id: I6ea12437bb6b4b9b8ee5c3b52d83509fa2437b2d Reviewed-on: https://go-review.googlesource.com/12243Reviewed-by:
Alex Brainman <alex.brainman@gmail.com> Reviewed-by:
Russ Cox <rsc@golang.org>
-
Robert Griesemer authored
See #10248 for details. Fixes #10248. Change-Id: I373545b2dca5d1da1c7149eb0a8f6c6dd8071a4c Reviewed-on: https://go-review.googlesource.com/10503Reviewed-by:
Russ Cox <rsc@golang.org> Reviewed-by:
Rob Pike <r@golang.org>
-
Alan Donovan authored
Change-Id: If0821a2af987b78ed8024b40d9ffa68032518b22 Reviewed-on: https://go-review.googlesource.com/12572Reviewed-by:
Robert Griesemer <gri@golang.org>
-
David Chase authored
Limit probe to ODOT/OIND/ODOTPTR for now; that works. Fixes #11790 Change-Id: I411271e702c5fe6ceb880ca47c7dacc37ffcbb6a Reviewed-on: https://go-review.googlesource.com/12532Reviewed-by:
Russ Cox <rsc@golang.org>
-
Didier Spezia authored
EncodeToken takes a Token (i.e. an interface{}) as a parameter, and expects a value of type StartElement, EndElement, CharData, Comment, ProcInst, or Directive. If a pointer is passed instead, or any type which does not match this list, the token is silently ignored. Added a default case in the type switch to issue a proper error when the type is invalid. The behavior could be later improved by allowing pointers to token to be accepted as well, but not for go1.5. Fixes #11719 Change-Id: Ifd13c1563450b474acf66d57669fdccba76c1949 Reviewed-on: https://go-review.googlesource.com/12252Reviewed-by:
Andrew Gerrand <adg@golang.org> Reviewed-by:
Russ Cox <rsc@golang.org>
-
Russ Cox authored
Foreign code can be arbitrarily aligned, so the function before it can have arbitrarily much padding. We can't call pcvalue on values in the padding. Fixes #11653. Change-Id: I7d57f813ae5a2409d1520fcc909af3eeef2da131 Reviewed-on: https://go-review.googlesource.com/12550Reviewed-by:
Rob Pike <r@golang.org>
-
Russ Cox authored
Fixes #11606. Change-Id: I70d38c22812c17119b998aad9c1c68e7cf74e98a Reviewed-on: https://go-review.googlesource.com/12524Reviewed-by:
Rob Pike <r@golang.org>
-
Russ Cox authored
We use 127.0.0.1 instead of localhost in Go networking tests. The reporter of #11774 has localhost defined to be 120.192.83.162, for reasons unknown. Also, if TestTraceSymbolize calls Fatalf (for example because Listen fails) then we need to stop the trace for future tests to work. See failure log in #11774. Fixes #11774. Change-Id: Iceddb03a72d31e967acd2d559ecb78051f9c14b7 Reviewed-on: https://go-review.googlesource.com/12521Reviewed-by:
Rob Pike <r@golang.org>
-
Russ Cox authored
Fixes #11681. Change-Id: I679d71ed25ac585af7d43611be01c1a0c4807871 Reviewed-on: https://go-review.googlesource.com/12554 Run-TryBot: Russ Cox <rsc@golang.org> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by:
Ian Lance Taylor <iant@golang.org>
-
Russ Cox authored
Fixes #11729. Change-Id: I6e5e23169ac1368afcbd016ed544a710aa045326 Reviewed-on: https://go-review.googlesource.com/12553Reviewed-by:
Rob Pike <r@golang.org>
-
Rob Pike authored
Also make the spelling consistent in asm.html Change-Id: Ifa751eee288fe0634cd317eb827f3e408b199620 Reviewed-on: https://go-review.googlesource.com/12501Reviewed-by:
Brad Fitzpatrick <bradfitz@golang.org> Reviewed-by:
Russ Cox <rsc@golang.org>
-
Brad Fitzpatrick authored
It wasn't working. The wrong variable was used. This would ideally have tests. It's also DEBUG. Fixes #11816 Change-Id: Iec42d229b81d78cece4ba5c73f3040e2356eb98f Reviewed-on: https://go-review.googlesource.com/12544 Run-TryBot: Brad Fitzpatrick <bradfitz@golang.org> Reviewed-by:
Ian Lance Taylor <iant@golang.org> TryBot-Result: Gobot Gobot <gobot@golang.org>
-
Paul Marks authored
When dialing with a relative Timeout instead of an absolute Deadline, the deadline function only makes sense if called before doing any time-consuming work. This change calls deadline exactly once, storing the result until the Dial operation completes. The partialDeadline implementation is reverted to the following patch set 3: https://go-review.googlesource.com/#/c/8768/3..4/src/net/dial.go Otherwise, when dialing a name with multiple IP addresses, or when DNS is slow, the recomputed deadline causes the total Timeout to exceed that requested by the user. Fixes #11796 Change-Id: I5e1f0d545f9e86a4e0e2ac31a9bd108849cf0fdf Reviewed-on: https://go-review.googlesource.com/12442 Run-TryBot: Paul Marks <pmarks@google.com> Run-TryBot: Mikio Hara <mikioh.mikioh@gmail.com> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by:
Brad Fitzpatrick <bradfitz@golang.org>
-
- 22 Jul, 2015 19 commits
-
-
Russ Cox authored
Fixes #10400. Change-Id: Ic486cb8af4c40660fd1a2e3d10986975acba3f19 Reviewed-on: https://go-review.googlesource.com/12537Reviewed-by:
Ian Lance Taylor <iant@golang.org>
-
Ian Lance Taylor authored
This is needed to handle vendor directories correctly. It was already done for the regular imports when the package was loaded, but not for the test-only imports. It would be nice to do this while loading the package, but that breaks the code that checks for direct references to vendor packages when running go test. This change is relatively contained. While we're at it, skip "C" test imports in go get. Fixes #11628. Fixes #11717. Change-Id: I9cc308cf45683e3ff905320c2b5cb45db7716846 Reviewed-on: https://go-review.googlesource.com/12488 Run-TryBot: Ian Lance Taylor <iant@golang.org> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by:
Russ Cox <rsc@golang.org>
-
Russ Cox authored
Fixes #11823. Change-Id: Ic949ccb9657478f8ca34fdf1a6fe88f57db69f24 Reviewed-on: https://go-review.googlesource.com/12535Reviewed-by:
Austin Clements <austin@google.com>
-
MathiasB authored
Updated Address.String so it restores quoted local parts, which wasn't done before. When parsing `<" "@example.com>`, the formatted string returned `< @example>`, which doens't match RFC 5322, since a space is not atext. Another example is `<"bob@valid"@example.com>` which returned `<bob@valid@example.com>`, which is completely invalid. I also added support for quotes and backslashes in a quoted local part. Besides formatting a parsed Address, the ParseAddress function also needed more testing and finetuning for special cases. Things like `<.john.doe@example.com>` and `<john..doe@example.com>` e.a. were accepted, but are invalid. I fixed those details and add tests for some other special cases. Fixes #10768 Change-Id: Ib0caf8ad603eb21e32fcb957a5f1a0fe5d1c6e6e Reviewed-on: https://go-review.googlesource.com/8724Reviewed-by:
Russ Cox <rsc@golang.org>
-
Russ Cox authored
Fixes #11665. Change-Id: I0897e8cf695434e77d14dcb1d96f21747edfe37c Reviewed-on: https://go-review.googlesource.com/12523Reviewed-by:
Ian Lance Taylor <iant@golang.org>
-
Russ Cox authored
Some routines run without and m or g and cannot invoke the race detector runtime. They must be opaque to the runtime. That used to be true because they were written in C. Now that they are written in Go, disable the race detector annotations for those functions explicitly. Add test. Fixes #10874. Change-Id: Ia8cc28d51e7051528f9f9594b75634e6bb66a785 Reviewed-on: https://go-review.googlesource.com/12534Reviewed-by:
Ian Lance Taylor <iant@golang.org>
-
Russ Cox authored
Same as we do for string symbols. Fixes #11583. Change-Id: Ia9264f6faf486697d987051b7f9851d37d8ad381 Reviewed-on: https://go-review.googlesource.com/12531Reviewed-by:
Josh Bleecher Snyder <josharian@gmail.com> Reviewed-by:
Ian Lance Taylor <iant@golang.org>
-
Russ Cox authored
Fixes #10842. Change-Id: I7de98f3073a47911863a252b7a74d8fdaa48c86f Reviewed-on: https://go-review.googlesource.com/12529Reviewed-by:
Brad Fitzpatrick <bradfitz@golang.org>
-
Ian Lance Taylor authored
In the past badsignal would crash the program. In https://golang.org/cl/10757044 badsignal was changed to call sigsend, to fix issue #3250. The effect of this was that when a non-Go thread received a signal, and os/signal.Notify was not being used to check for occurrences of the signal, the signal was ignored. This changes the code so that if os/signal.Notify is not being used, then the signal handler is reset to what it was, and the signal is raised again. This lets non-Go threads handle the signal as they wish. In particular, it means that a segmentation violation in a non-Go thread will ordinarily crash the process, as it should. Fixes #10139. Update #11794. Change-Id: I2109444aaada9d963ad03b1d071ec667760515e5 Reviewed-on: https://go-review.googlesource.com/12503Reviewed-by:
Russ Cox <rsc@golang.org> Run-TryBot: Ian Lance Taylor <iant@golang.org>
-
Konstantin Shaposhnikov authored
This change fixes resolution of secure (https) repo URL for git.apache.org Git repositories. E.g. the correct repo URL for git.apache.org/thrift.git/lib/go/thrift is https://git.apache.org/thrift.git, not https://git.apache.org/thrift Fixes #10797 Change-Id: I67d5312ad8620eb780e42c2e002c8f286f60645a Reviewed-on: https://go-review.googlesource.com/10092Reviewed-by:
Andrew Gerrand <adg@golang.org> Reviewed-by:
Russ Cox <rsc@golang.org>
-
Robert Griesemer authored
Fixes #11523. Change-Id: I172f6facd555a1c6db76f25d5097343c20dea59a Reviewed-on: https://go-review.googlesource.com/12507Reviewed-by:
Alan Donovan <adonovan@google.com> Reviewed-by:
Russ Cox <rsc@golang.org>
-
Russ Cox authored
It's a bad test and it's worst on uniprocessors. Fixes #11143. Change-Id: I0164231ada294788d7eec251a2fc33e02a26c13b Reviewed-on: https://go-review.googlesource.com/12522Reviewed-by:
Brad Fitzpatrick <bradfitz@golang.org>
-
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>
-