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

(v0.22.0) Add host class package to anon class name for 11+ #10482

Conversation

mikezhang1234567890
Copy link
Contributor

Reflection fails for anonymous classes with a host class in a module which
exports only certain packages, since in OpenJ9, the anonymous class name does
not include package info. In the reference implementation, anonymous classes
do have package info in their name, so this commit changes OpenJ9 behaviour to
match that.

Signed-off-by: Mike Zhang mike.h.zhang@ibm.com

Port of #10382 to 0.22.0

Reflection fails for anonymous classes with a host class in a module which
exports only certain packages, since in OpenJ9, the anonymous class name does
not include package info. In the reference implementation, anonymous classes
do have package info in their name, so this commit changes OpenJ9 behaviour to
match that.

Signed-off-by: Mike Zhang <mike.h.zhang@ibm.com>
@pshipton pshipton added this to the Release 0.22 (Java 15) milestone Aug 27, 2020
@DanHeidinga DanHeidinga self-assigned this Aug 28, 2020
Copy link
Member

@DanHeidinga DanHeidinga left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

merged into the main branch already

@DanHeidinga DanHeidinga merged commit 5bb6fc7 into eclipse-openj9:v0.22.0-release Aug 28, 2020
@mikezhang1234567890 mikezhang1234567890 deleted the v0.22.0-release branch June 13, 2022 16:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants