all: change github.com issue links to golang.org

The go repository contains a mix of github.com/golang/go/issues/xxxxx
and golang.org/issues/xxxxx URLs for references to issues in the issue
tracker. We should use one for consistency, and golang.org is preferred
in case the project moves the issue tracker in the future.

This reasoning is taken from a comment Sam Whited left on a CL I
recently opened: https://go-review.googlesource.com/c/go/+/73890.

In that CL I referenced an issue using its github.com URL, because other
tests in the file I was changing contained references to issues using
their github.com URL. Sam Whited left a comment on the CL stating I
should change it to the golang.org URL.

If new code is intended to reference issues via golang.org and not
github.com, existing code should be updated so that precedence exists
for contributors who are looking at the existing code as a guide for the
code they should write.

Change-Id: I3b9053fe38a1c56fc101a8b7fd7b8f310ba29724
Reviewed-on: https://go-review.googlesource.com/75673
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>

diff --git a/misc/cgo/testshared/src/depBase/dep.go b/misc/cgo/testshared/src/depBase/dep.go
index 9f86710db01a4438fe4ab7fb7e8f3b13829592f5..569c210aa14dc8afc9bc7e8116bc0bcad9fdd4f1 100644 (file)
--- a/misc/cgo/testshared/src/depBase/dep.go
+++ b/misc/cgo/testshared/src/depBase/dep.go
@@ -22,7 +22,7 @@ type Dep struct {
 func (d *Dep) Method() int {
        // This code below causes various go.itab.* symbols to be generated in
        // the shared library. Similar code in ../exe/exe.go results in
-       // exercising https://github.com/golang/go/issues/17594
+       // exercising https://golang.org/issues/17594
        reflect.TypeOf(os.Stdout).Elem()
        return 10
 }

diff --git a/misc/cgo/testshared/src/exe/exe.go b/misc/cgo/testshared/src/exe/exe.go
index 84302a811f0919b9035b8da22814581af1d122a0..bd864d88ad8435f15769e759375759e99daf20ea 100644 (file)
--- a/misc/cgo/testshared/src/exe/exe.go
+++ b/misc/cgo/testshared/src/exe/exe.go
@@ -25,7 +25,7 @@ func main() {
        defer depBase.ImplementedInAsm()
        // This code below causes various go.itab.* symbols to be generated in
        // the executable. Similar code in ../depBase/dep.go results in
-       // exercising https://github.com/golang/go/issues/17594
+       // exercising https://golang.org/issues/17594
        reflect.TypeOf(os.Stdout).Elem()
        runtime.GC()
        depBase.V = depBase.F() + 1

diff --git a/src/crypto/elliptic/elliptic_test.go b/src/crypto/elliptic/elliptic_test.go
index 2c0f2440f74bbdb4938f3adf9a57ada3f276b523..55c6e894b01d46d9f1834b2ff24427e11ca6fbd8 100644 (file)
--- a/src/crypto/elliptic/elliptic_test.go
+++ b/src/crypto/elliptic/elliptic_test.go
@@ -581,7 +581,7 @@ func TestP224Overflow(t *testing.T) {
        }
 }
 
-// See https://github.com/golang/go/issues/20482
+// See https://golang.org/issues/20482
 func TestUnmarshalToLargeCoordinates(t *testing.T) {
        curve := P256()
        p := curve.Params().P

diff --git a/src/crypto/x509/sec1_test.go b/src/crypto/x509/sec1_test.go
index 55b76d6c38160464040fcb33ac15b0854bcea5e8..573c937cafdf8b8fe5e202a86ef8da6b2b20189a 100644 (file)
--- a/src/crypto/x509/sec1_test.go
+++ b/src/crypto/x509/sec1_test.go
@@ -18,7 +18,7 @@ var ecKeyTests = []struct {
        //   openssl ecparam -genkey -name secp384r1 -outform PEM
        {"3081a40201010430bdb9839c08ee793d1157886a7a758a3c8b2a17a4df48f17ace57c72c56b4723cf21dcda21d4e1ad57ff034f19fcfd98ea00706052b81040022a16403620004feea808b5ee2429cfcce13c32160e1c960990bd050bb0fdf7222f3decd0a55008e32a6aa3c9062051c4cba92a7a3b178b24567412d43cdd2f882fa5addddd726fe3e208d2c26d733a773a597abb749714df7256ead5105fa6e7b3650de236b50", true},
        // This key was generated by GnuTLS and has illegal zero-padding of the
-       // private key. See https://github.com/golang/go/issues/13699.
+       // private key. See https://golang.org/issues/13699.
        {"3078020101042100f9f43a04b9bdc3ab01f53be6df80e7a7bc3eaf7b87fc24e630a4a0aa97633645a00a06082a8648ce3d030107a1440342000441a51bc318461b4c39a45048a16d4fc2a935b1ea7fe86e8c1fa219d6f2438f7c7fd62957d3442efb94b6a23eb0ea66dda663dc42f379cda6630b21b7888a5d3d", false},
        // This was generated using an old version of OpenSSL and is missing a
        // leading zero byte in the private key that should be present.

diff --git a/src/database/sql/convert_test.go b/src/database/sql/convert_test.go
index c198177760ac32f1c6941269dfed76495fa482bd..47098c81ec1700a3c494b31f2964f03dc22215c2 100644 (file)
--- a/src/database/sql/convert_test.go
+++ b/src/database/sql/convert_test.go
@@ -398,7 +398,7 @@ func TestRawBytesAllocs(t *testing.T) {
        }
 }
 
-// https://github.com/golang/go/issues/13905
+// https://golang.org/issues/13905
 func TestUserDefinedBytes(t *testing.T) {
        type userDefinedBytes []byte
        var u userDefinedBytes

diff --git a/src/encoding/json/encode_test.go b/src/encoding/json/encode_test.go
index 3fda6a0c719d431af2f04fec90ba53bca652c80a..db89d113da398e92100abfd8f2973c0fac4b7927 100644 (file)
--- a/src/encoding/json/encode_test.go
+++ b/src/encoding/json/encode_test.go
@@ -943,7 +943,7 @@ func TestMarshalRawMessageValue(t *testing.T) {
                //
                // The tests below marked with Issue6458 used to generate "ImZvbyI=" instead "foo".
                // This behavior was intentionally changed in Go 1.8.
-               // See https://github.com/golang/go/issues/14493#issuecomment-255857318
+               // See https://golang.org/issues/14493#issuecomment-255857318
                {rawText, `"foo"`, true}, // Issue6458
                {&rawText, `"foo"`, true},
                {[]interface{}{rawText}, `["foo"]`, true},  // Issue6458

diff --git a/src/encoding/xml/read_test.go b/src/encoding/xml/read_test.go
index ba203cd98b55a6a24531477e47296b2dc625b384..8c2e70fa22ee68e06956dae82a3fafa1c2b779a1 100644 (file)
--- a/src/encoding/xml/read_test.go
+++ b/src/encoding/xml/read_test.go
@@ -819,7 +819,7 @@ const (
 `
 )
 
-// github.com/golang/go/issues/13417
+// golang.org/issues/13417
 func TestUnmarshalEmptyValues(t *testing.T) {
        // Test first with a zero-valued dst.
        v := new(Parent)

diff --git a/src/image/png/reader_test.go b/src/image/png/reader_test.go
index eb792ea80613f7af5754095786abe3e938f5c763..66bcfcb437eb2a208e49804f31f70aee87b51dec 100644 (file)
--- a/src/image/png/reader_test.go
+++ b/src/image/png/reader_test.go
@@ -589,7 +589,7 @@ func TestUnknownChunkLengthUnderflow(t *testing.T) {
 }
 
 func TestGray8Transparent(t *testing.T) {
-       // These bytes come from https://github.com/golang/go/issues/19553
+       // These bytes come from https://golang.org/issues/19553
        m, err := Decode(bytes.NewReader([]byte{
                0x89, 0x50, 0x4e, 0x47, 0x0d, 0x0a, 0x1a, 0x0a, 0x00, 0x00, 0x00, 0x0d, 0x49, 0x48, 0x44, 0x52,
                0x00, 0x00, 0x00, 0x0f, 0x00, 0x00, 0x00, 0x0b, 0x08, 0x00, 0x00, 0x00, 0x00, 0x85, 0x2c, 0x88,
@@ -650,7 +650,7 @@ func TestGray8Transparent(t *testing.T) {
 }
 
 func TestDimensionOverflow(t *testing.T) {
-       // These bytes come from https://github.com/golang/go/issues/22304
+       // These bytes come from https://golang.org/issues/22304
        //
        // It encodes a 2147483646 × 2147483646 (i.e. 0x7ffffffe × 0x7ffffffe)
        // NRGBA image. The (width × height) per se doesn't overflow an int64, but

diff --git a/src/net/hosts_test.go b/src/net/hosts_test.go
index 5d6c9cfe190a3bf3f5b8aec115b9ec54b1397b57..f850e2fccfd2baf03e9be8ee8ba02dbdd5555a09 100644 (file)
--- a/src/net/hosts_test.go
+++ b/src/net/hosts_test.go
@@ -150,7 +150,7 @@ func testStaticAddr(t *testing.T, hostsPath string, ent staticHostEntry) {
 
 func TestHostCacheModification(t *testing.T) {
        // Ensure that programs can't modify the internals of the host cache.
-       // See https://github.com/golang/go/issues/14212.
+       // See https://golang.org/issues/14212.
        defer func(orig string) { testHookHostsPath = orig }(testHookHostsPath)
 
        testHookHostsPath = "testdata/ipv4-hosts"

diff --git a/src/net/port.go b/src/net/port.go
index 8e1321afa44b2db37c286e76ae15fd1fd43b6cd4..9016a19831214c34994185e21f404723b372f628 100644 (file)
--- a/src/net/port.go
+++ b/src/net/port.go
@@ -9,7 +9,7 @@ package net
 // parse service as a non-decimal integer when needsLookup is true.
 //
 // Some system resolvers will return a valid port number when given a number
-// over 65536 (see https://github.com/golang/go/issues/11715). Alas, the parser
+// over 65536 (see https://golang.org/issues/11715). Alas, the parser
 // can't bail early on numbers > 65536. Therefore reasonably large/small
 // numbers are parsed in full and rejected if invalid.
 func parsePort(service string) (port int, needsLookup bool) {

diff --git a/src/os/stat_windows.go b/src/os/stat_windows.go
index e7a4e0d54af1bcc5823769d6875d6a0c231d97b5..71dea3d48fe4d917b09be7ec34c3757bbec3bd1e 100644 (file)
--- a/src/os/stat_windows.go
+++ b/src/os/stat_windows.go
@@ -69,7 +69,7 @@ func Stat(name string) (FileInfo, error) {
        if err != nil {
                return nil, &PathError{"Stat", name, err}
        }
-       // Apparently (see https://github.com/golang/go/issues/19922#issuecomment-300031421)
+       // Apparently (see https://golang.org/issues/19922#issuecomment-300031421)
        // GetFileAttributesEx is fastest approach to get file info.
        // It does not work for symlinks. But symlinks are rare,
        // so try GetFileAttributesEx first.

diff --git a/src/sync/atomic/value.go b/src/sync/atomic/value.go
index 1fc1f681f2024ca051dead986f299c9b480527ff..ac5a9a5e7228f72e0b807aaab51548be86857d17 100644 (file)
--- a/src/sync/atomic/value.go
+++ b/src/sync/atomic/value.go
@@ -90,7 +90,7 @@ func runtime_procUnpin()
 // noCopy may be embedded into structs which must not be copied
 // after the first use.
 //
-// See https://github.com/golang/go/issues/8005#issuecomment-190753527
+// See https://golang.org/issues/8005#issuecomment-190753527
 // for details.
 type noCopy struct{}
 

diff --git a/src/sync/cond.go b/src/sync/cond.go
index 14e2f6b24d4ad53ce603045a7dae3ffd2206506b..3dcbf1c3512f30d48c8e4758398cc81d3f04a3cc 100644 (file)
--- a/src/sync/cond.go
+++ b/src/sync/cond.go
@@ -89,7 +89,7 @@ func (c *copyChecker) check() {
 // noCopy may be embedded into structs which must not be copied
 // after the first use.
 //
-// See https://github.com/golang/go/issues/8005#issuecomment-190753527
+// See https://golang.org/issues/8005#issuecomment-190753527
 // for details.
 type noCopy struct{}