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

Provide method name when constructing VarHandleInvokeHandle #4458

Merged
merged 1 commit into from
Feb 11, 2019

Conversation

pdbain-ibm
Copy link
Contributor

Fixes #3180

Signed-off-by: Peter Bain peter_bain@ca.ibm.com

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.

lgtm

@DanHeidinga
Copy link
Member

@pdbain-ibm Can you fix the copyright date?

@DanHeidinga DanHeidinga self-assigned this Jan 25, 2019
@DanHeidinga
Copy link
Member

Can you also add MethodHandleInfo tests for all the VarHandle APIs? Something similar to https://github.com/eclipse/openj9/blob/2f295ea91cdd97ff37eb5d164fc63014ea718bc7/test/functional/Java8andUp/src/org/openj9/test/java/lang/invoke/Test_MethodHandleInfo.java but applicable to Java 11+.

That can be done in a separate PR

Fixes eclipse-openj9#3180

Signed-off-by: Peter Bain <peter_bain@ca.ibm.com>
@pdbain-ibm
Copy link
Contributor Author

Copyright fixed, issue for test case #4459.

@DanHeidinga
Copy link
Member

Jenkins test sanity zlinux jdk11

@DanHeidinga DanHeidinga merged commit a38f641 into eclipse-openj9:master Feb 11, 2019
pdbain-ibm added a commit to pdbain-ibm/openj9 that referenced this pull request Apr 2, 2019
Test for eclipse-openj9#3180

This depends on eclipse-openj9#4458

[ci skip]

Signed-off-by: Peter Bain <peter_bain@ca.ibm.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants