[lworld] RFR: 8363846: [lworld] Make Class.isIdentityClass() non-native [v2]

Coleen Phillimore coleenp at openjdk.org
Wed Jul 23 15:09:11 UTC 2025


On Wed, 23 Jul 2025 14:15:54 GMT, Roger Riggs <rriggs at openjdk.org> wrote:

>> 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.

yes, that's surprising it passed.  Oh, I don't know how these work.  None of the other tests check for !enablePreview in this directory.

-------------

PR Review Comment: https://git.openjdk.org/valhalla/pull/1514#discussion_r2225897395


More information about the valhalla-dev mailing list