[lworld] RFR: 8237073: [lworld] Need special handling of jlO constructor invocation [v2]
Srikanth Adayapalam
sadayapalam at openjdk.java.net
Mon Jul 19 05:54:29 UTC 2021
On Fri, 16 Jul 2021 18:16:50 GMT, Mandy Chung <mchung at openjdk.org> wrote:
>> Srikanth Adayapalam has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains three additional commits since the last revision:
>>
>> - Incorporate review comments from Mandy Chung
>> - Merge branch 'lworld' into JDK-8237073
>> - 8237073: [lworld] Need special handling of jlO constructor invocation
>
> test/jdk/jdk/dynalink/TrustedDynamicLinkerFactoryTest.java line 196:
>
>> 194: MethodHandles.publicLookup(), GET_PROPERTY, mt)));
>> 195: Assert.assertFalse(reachedPrelinkTransformer[0]);
>> 196: Assert.assertEquals(cs.getTarget().invoke(new Object(), "class"), java.util.Objects.newIdentity().getClass());
>
> Similar to BeanLinkerTest.java, it's clearer to define a TestObject class to replace "new Object()" in this test.
Done!
-------------
PR: https://git.openjdk.java.net/valhalla/pull/481
More information about the valhalla-dev
mailing list