On Fri, 18 Sep 2026 08:01:30 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). src/jdk.net/share/classes/jdk/net/Sockets.java line 272: > 270: } > 271: > 272: private static void checkValueType(Object value, Class<?> type) { Can this method be removed too? ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/32943#discussion_r4044995250
