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) Hidden class implementation 4 #10520

Merged
merged 1 commit into from Sep 3, 2020

Conversation

hangshao0
Copy link
Contributor

  1. Add checks for class bytes for hidden classes in MethodHanles.
  2. Do bytecode verification for hidden classes.
  3. Do not allow record and enum to be defined as hidden classes.
  4. Fix APIs for hidden classes in Class.java.

Closes #9328

Signed-off-by: Hang Shao hangshao@ca.ibm.com

1. Add checks for class bytes for hidden classes in MethodHanles.
2. Do bytecode verification for hidden classes.
3. Do not allow record and enum to be defined as hidden classes.
4. Fix APIs for hidden classes in Class.java.

Closes eclipse-openj9#9328

Signed-off-by: Hang Shao <hangshao@ca.ibm.com>
@hangshao0 hangshao0 changed the title Hidden class implementation 4 (v0.22.0) Hidden class implementation 4 Sep 3, 2020
@hangshao0
Copy link
Contributor Author

Cherry picked on c464059 + updated the copyright date of vrfyhelp.c

@gacholio
Copy link
Contributor

gacholio commented Sep 3, 2020

Closing this and reopening #10517

@gacholio gacholio closed this Sep 3, 2020
@gacholio
Copy link
Contributor

gacholio commented Sep 3, 2020

Nope, guess that's not going to work.

@gacholio gacholio reopened this Sep 3, 2020
@gacholio
Copy link
Contributor

gacholio commented Sep 3, 2020

jenkins test sanity zlinux jdk15

@gacholio
Copy link
Contributor

gacholio commented Sep 3, 2020

Builds don't work properly from a branch, it seems, so I'll just go ahead and merge this.

@gacholio gacholio merged commit 2d2f8b8 into eclipse-openj9:v0.22.0-release Sep 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants