Skip to content

Add simple class name check in checkInnerClassAttrOfEnclosingClass() - #19126

Merged
tajila merged 1 commit into
eclipse-openj9:masterfrom
hangshao0:ICCE
Mar 15, 2024
Merged

Add simple class name check in checkInnerClassAttrOfEnclosingClass()#19126
tajila merged 1 commit into
eclipse-openj9:masterfrom
hangshao0:ICCE

Conversation

@hangshao0

Copy link
Copy Markdown
Contributor

Anon class (simple name is null) cannot be referenced by name from the inner class attribute of its enclosing class. In this case, there is no need to search for the class name in the inner class attribute.

Closes #19016

Anon class (simple name is null) cannot be referenced by name from the
inner class attribute of its enclosing class. In this case, there is no
need to search for the class name in the inner class attribute.

Closes eclipse-openj9#19016

Signed-off-by: Hang Shao <hangshao@ca.ibm.com>
@hangshao0
hangshao0 requested review from ChengJin01 and tajila March 12, 2024 18:30

@ChengJin01 ChengJin01 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM.

@tajila

tajila commented Mar 14, 2024

Copy link
Copy Markdown
Contributor

jenkins test sanity alinux64 jdk21

@tajila

tajila commented Mar 14, 2024

Copy link
Copy Markdown
Contributor

jenkins test sanity plinux jdk11

@hangshao0

Copy link
Copy Markdown
Contributor Author

The only failure in the PR build is a jitserver test which failed due to

[ERR] curl: (7) Failed to connect to localhost port 43895: Connection refused

It is not related to this change.

@tajila
tajila merged commit 6dbe197 into eclipse-openj9:master Mar 15, 2024
@hangshao0
hangshao0 deleted the ICCE branch September 26, 2024 18:45
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.

Hotspot-8 normal execution, but OpenJ9 thrown IncompatibleClassChangeError

3 participants