On Tue, 22 Sep 2026 00:34:54 GMT, Lee Jiwon <[email protected]> wrote:
>> Before this change, Sockets.supportedOptions(Class<?>) used a hard-coded >> option map created during class initialization. The reported set could >> differ from the options supported by an actual socket instance. >> >> This change creates an instance of the requested socket type for each call >> and returns its supportedOptions() result. The temporary socket is closed >> after use, and jdk.net.Sockets does not cache the result. The hard-coded >> option map and helper code used only to build that map are removed. >> >> Testing: >> >> * make test TEST=test/jdk/jdk/net/Sockets/SupportedOptions.java >> * make test TEST=test/jdk/jdk/net/Sockets >> * make test TEST=jdk_net — 1,082 tests passed >> >> --------- >> - [x] I confirm that I make this contribution in accordance with the >> [OpenJDK Interim AI Policy](https://openjdk.org/legal/ai). > > Lee Jiwon has updated the pull request incrementally with one additional > commit since the last revision: > > Remove redundant IP_TOS test from SupportedOptions LGTM ------------- Marked as reviewed by michaelm (Reviewer). PR Review: https://git.openjdk.org/jdk/pull/32943#pullrequestreview-5275549518
