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

cmd/gc: wrong embedded invisible method called #3146

Closed
griesemer opened this issue Feb 28, 2012 · 2 comments
Closed

cmd/gc: wrong embedded invisible method called #3146

griesemer opened this issue Feb 28, 2012 · 2 comments
Milestone

Comments

@griesemer
Copy link
Contributor

What steps will reproduce the problem?
- see test case test/bugs/424.dir/*
- http://golang.org/cl/5695083

What is the expected output? What do you see instead?

- In all cases, the same local method main.localT.m should be called. However, in the
case of type myT3 (which is the same as myT2 except that the embedded fields are in a
different order), the wrong non-exported method m from package lib is invoked instead.

This is clearly a compiler/runtime bug. The spec is not clear on this topic but the
issue remains valid. There are related issues 1536, 1550, and 1551 (main issue).
@rsc
Copy link
Contributor

rsc commented Feb 28, 2012

Comment 1:

Nice bug!

@rsc
Copy link
Contributor

rsc commented Mar 7, 2012

Comment 3:

This issue was closed by revision 987a580.

Status changed to Fixed.

@rsc rsc added this to the Go1 milestone Apr 10, 2015
@golang golang locked and limited conversation to collaborators Jun 24, 2016
@rsc rsc removed their assignment Jun 22, 2022
This issue was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants