[lworld] RFR: 8363846: [lworld] Make Class.isIdentityClass() non-native [v2]
Roger Riggs
rriggs at openjdk.org
Wed Jul 23 14:18:06 UTC 2025
On Wed, 23 Jul 2025 14:10:18 GMT, Roger Riggs <rriggs at openjdk.org> wrote:
>> Coleen Phillimore has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Add a test for not enable preview.
>
> test/jdk/valhalla/valuetypes/IsIdentityClassTest.java line 68:
>
>> 66: assertFalse(ValueClass.isFlatArray(array0));
>> 67: }
>> 68: assertFalse(Integer.class.isIdentity(), "Integer is not an IDENTITY type");
>
> This test should be sensitive to whether enable-preview is true.
> It should have failed in the second run.
> But due to the TEST.properties (enablePreview = true) it overrides the command line in the test.
> Removing enablePreview in TEST.properties is the right fix for that but then another test needs a fix.
> The RecursiveValueClass test will need to have @enablePreview added to each of its test runs.
I'll fixup TEST.properties and the other test in a separate PR.
-------------
PR Review Comment: https://git.openjdk.org/valhalla/pull/1514#discussion_r2225754960
More information about the valhalla-dev
mailing list