On Thu, 8 May 2025 17:47:12 GMT, Petr Portnov | PROgrm_JARvis
<[email protected]> wrote:
>> Andrew Haley has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Fix merge
>
> src/java.base/share/classes/java/lang/ScopedValue.java line 613:
>
>> 611: */
>> 612: public T orElse(T other) {
>> 613: Objects.requireNonNull(other);
>
> Shouldn't the NPE be specified in the Javadoc?
In the class specification, right above the API notes:
* <p> Unless otherwise specified, passing a {@code null} argument to a method
in this
* class will cause a {@link NullPointerException} to be thrown.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/24923#discussion_r2080203356
