On Mon, 13 Apr 2026 07:29:39 GMT, Volkan Yazici <[email protected]> wrote:

>> test/jdk/java/net/SocketOption/ImmutableOptions.java line 70:
>> 
>>> 68:         assertThrows(UnsupportedOperationException.class,
>>> 69:                 () -> socket.supportedOptions().clear());
>>> 70:     }
>> 
>> Here and below, what about moving the `supportedOptions()` call outside the 
>> `assertThrows` to ensure that it doesn't unexpectedly throw the 
>> `UnsupportedOperationException`?
>
> Agreed.

Done

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

PR Review Comment: https://git.openjdk.org/jdk/pull/30681#discussion_r3073378523

Reply via email to