[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