> Can I please get a review of this change which proposes to respecify the 2 > `java.net.Socket` constructors that allow construction of UDP sockets? This > addresses https://bugs.openjdk.org/browse/JDK-8356154. > > As noted in that JBS issue, in Java 23 we deprecated for removal the 2 > `Socket` constructors that allowed UDP socket creation. The plan continues to > be to remove those constructors. Before removing those, in order to allow for > applications to notice this deprecation, these constructors are now being > respecified to throw an `IllegalArgumentException` when the `stream` > parameter is `false`. > > I will create a CSR once we settle on these changes. > > tier1 through tier8 tests have been run with this change and no related > failures have been noticed.
Jaikiran Pai has updated the pull request incrementally with one additional commit since the last revision: additional changes to SocketImpl.create(...) API javadoc based on review suggestions ------------- Changes: - all: https://git.openjdk.org/jdk/pull/25031/files - new: https://git.openjdk.org/jdk/pull/25031/files/ad880d50..8e329ff6 Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk&pr=25031&range=03 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=25031&range=02-03 Stats: 10 lines in 1 file changed: 0 ins; 4 del; 6 mod Patch: https://git.openjdk.org/jdk/pull/25031.diff Fetch: git fetch https://git.openjdk.org/jdk.git pull/25031/head:pull/25031 PR: https://git.openjdk.org/jdk/pull/25031
