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
cmd/compile: improve "have X want Y" error with additional file and line information #42841
Comments
Seems overlooked by triage? |
@griesemer Should this stay in the Go1.18 milestone or should we move it to Backlog? |
I think this is either done or mostly done (or pending CL). We should keep this for 1.18; I will take care of it. |
Change https://golang.org/cl/373634 mentions this issue: |
I've mailed a CL for this per https://golang.org/cl/373634. |
…face satisfaction With this change, we shall now see: *myS does not implement S (wrong type for DoSomething method) have DoSomething() (string, error) at ./main.go:9:14 want DoSomething() (int, error) instead of previously: *myS does not implement S (wrong type for DoSomething method) have DoSomething() (string, error) want DoSomething() (int, error) Fixes golang#42841 Fixes golang#45813 Change-Id: I66990929e39b0d36f2e91da0d92f60586a9b84e5 Reviewed-on: https://go-review.googlesource.com/c/go/+/373634 Trust: Robert Findley <rfindley@google.com> Trust: Emmanuel Odeke <emmanuel@orijtech.com> Run-TryBot: Emmanuel Odeke <emmanuel@orijtech.com> TryBot-Result: Gopher Robot <gobot@golang.org> Reviewed-by: Robert Griesemer <gri@golang.org>
What version of Go are you using (
go version
)?Does this issue reproduce with the latest release?
Yes
What did you do?
What did you expect to see?
Show also line information (
foo.go:123:57
) for have line. The implementation might be in completely different file. In modern IDEs you simply can click the file information line to navigate to referenced file and line quickly.want could also have line information to referenced interface definition line?
What did you see instead?
The text was updated successfully, but these errors were encountered: