RFR: 8351565: Implement JEP 502: Stable Values (Preview) [v78]

Per Minborg pminborg at openjdk.org
Wed Apr 30 06:30:13 UTC 2025


On Tue, 29 Apr 2025 14:51:08 GMT, Jorn Vernee <jvernee at openjdk.org> wrote:

>> Per Minborg has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Make public constuctor private
>
> test/jdk/java/lang/StableValue/StableFunctionTest.java line 79:
> 
>> 77:     void factoryInvariants(Set<Value> inputs) {
>> 78:         assertThrows(NullPointerException.class, () -> StableValue.function(null, MAPPER));
>> 79:         assertThrows(NullPointerException.class, () -> StableValue.function(inputs, null));
> 
> I think you also need a case for `null` elements in `inputs`

There is a separate `@Test void nullKeys()` that takes care of that case.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/23972#discussion_r2067935980


More information about the core-libs-dev mailing list