Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

go/types: interface.Complete panics for interfaces with duplicate methods [1.21 backport] #61743

Closed
gopherbot opened this issue Aug 3, 2023 · 2 comments
Labels
CherryPickApproved Used during the release process for point releases release-blocker
Milestone

Comments

@gopherbot
Copy link
Contributor

@findleyr requested issue #61737 to be considered for backport to the next 1.21 minor release.

@gopherbot please backport to Go 1.21. This is a deterministic panic when correctly using the go/types API.

@gopherbot gopherbot added the CherryPickCandidate Used during the release process for point releases label Aug 3, 2023
@gopherbot gopherbot added this to the Go1.21.1 milestone Aug 3, 2023
@gopherbot
Copy link
Contributor Author

Change https://go.dev/cl/515636 mentions this issue: [release-branch.go1.21] go/types, types2: don't panic during interface completion

@dmitshur dmitshur added CherryPickApproved Used during the release process for point releases and removed CherryPickCandidate Used during the release process for point releases labels Aug 16, 2023
gopherbot pushed a commit that referenced this issue Aug 17, 2023
…e completion

It should be possible for the importer to construct an invalid
interface, as would have been produced by type checking.

Updates #61737
Fixes #61743

Change-Id: I72e063f4f1a6205d273a623acce2ec08c34c3cc2
Reviewed-on: https://go-review.googlesource.com/c/go/+/515555
Reviewed-by: Robert Griesemer <gri@google.com>
Auto-Submit: Robert Findley <rfindley@google.com>
Reviewed-by: Olif Oftimis <oftimisolif@gmail.com>
Run-TryBot: Robert Findley <rfindley@google.com>
TryBot-Result: Gopher Robot <gobot@golang.org>
(cherry picked from commit d2ee782)
Reviewed-on: https://go-review.googlesource.com/c/go/+/515636
Run-TryBot: Dmitri Shuralyov <dmitshur@golang.org>
Auto-Submit: Dmitri Shuralyov <dmitshur@google.com>
@gopherbot
Copy link
Contributor Author

Closed by merging 4e34f2e to release-branch.go1.21.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CherryPickApproved Used during the release process for point releases release-blocker
Projects
None yet
Development

No branches or pull requests

3 participants