Repository navigation
Conversation
The docs for Interface.Method and Interface.ExplicitMethod say the methods are ordered by their unique Id. Since CL 461677 they are sorted by Object.cmp instead: exported methods first, then by name and, for non-exported methods with the same name, by package path. The two orders differ when an interface has unexported methods from more than one package. Describe the order that is implemented. Fixes golang#81875
|
This PR (HEAD: 9cefc6e) has been imported to Gerrit for code review. Please visit Gerrit at https://go-review.googlesource.com/c/go/+/847325. Important tips:
|
|
Message from Gopher Robot: Patch Set 1: Congratulations on opening your first change. Thank you for your contribution! Next steps: Most changes in the Go project go through a few rounds of revision. This can be During May-July and Nov-Jan the Go project is in a code freeze, during which Please don’t reply on this GitHub thread. Visit golang.org/cl/847325. |
|
Message from Robert Griesemer: Patch Set 1: Auto-Submit+1 Code-Review+2 Commit-Queue+1 Please don’t reply on this GitHub thread. Visit golang.org/cl/847325. |
|
Message from golang-scoped@luci-project-accounts.iam.gserviceaccount.com: Patch Set 1: Dry run: CV is trying the patch. Bot data: {"action":"start","triggered_at":"2026-10-09T01:46:36Z","revision":"b7f4015d776215ab7601baaf6a740a0f2e2c10de"} Please don’t reply on this GitHub thread. Visit golang.org/cl/847325. |
|
Message from Robert Griesemer: Patch Set 1: -Commit-Queue (Performed by <GERRIT_ACCOUNT_60063> on behalf of <GERRIT_ACCOUNT_7392>) Please don’t reply on this GitHub thread. Visit golang.org/cl/847325. |
|
Message from golang-scoped@luci-project-accounts.iam.gserviceaccount.com: Patch Set 1: This CL has passed the run Please don’t reply on this GitHub thread. Visit golang.org/cl/847325. |
|
Message from golang-scoped@luci-project-accounts.iam.gserviceaccount.com: Patch Set 1: LUCI-TryBot-Result+1 Please don’t reply on this GitHub thread. Visit golang.org/cl/847325. |
The docs for Interface.Method and Interface.ExplicitMethod say the methods are ordered by their unique Id. Since CL 461677 they are sorted by Object.cmp instead: exported methods first, then by name and, for non-exported methods with the same name, by package path. The two orders differ when an interface has unexported methods from more than one package. Describe the order that is implemented. Fixes #81875 Change-Id: Ia00712747b61355de1cb2fa2fdc2b036af64e90e GitHub-Last-Rev: 9cefc6e GitHub-Pull-Request: #82083 Reviewed-on: https://go-review.googlesource.com/c/go/+/847325 Auto-Submit: Robert Griesemer <gri@google.com> LUCI-TryBot-Result: golang-scoped@luci-project-accounts.iam.gserviceaccount.com <golang-scoped@luci-project-accounts.iam.gserviceaccount.com> Reviewed-by: Robert Griesemer <gri@google.com> Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
|
This PR is being closed because golang.org/cl/847325 has been merged. |
The docs for Interface.Method and Interface.ExplicitMethod say the
methods are ordered by their unique Id. Since CL 461677 they are
sorted by Object.cmp instead: exported methods first, then by name
and, for non-exported methods with the same name, by package path.
The two orders differ when an interface has unexported methods from
more than one package. Describe the order that is implemented.
Fixes #81875