RFR: 8216984: Deprecate for removal Socket constructors to create UDP sockets [v2]

Jaikiran Pai jpai at openjdk.org
Wed May 29 05:51:27 UTC 2024


> Can I please get a review of this change which marks 2 constructors on `java.net.Socket` as deprecated for removal?
> 
> As noted in https://bugs.openjdk.org/browse/JDK-8216984 these 2 `Socket` constructors, which allow for `stream=false` to construct a UDP socket, have been deprecated since several releases (starting Java 1.1). `java.net.DatagramSocket` has been a standard API for dealing with UDP sockets as noted in the already existing deprecation note on these constructors.
> 
> The commit in this PR marks these constructs as deprecated for removal to allow for their removal in some future release.
> 
> No new tests have been added for this change. tier1, tier2 and tier3 testing is currently in progress.
> 
> A CSR for this change is available at https://bugs.openjdk.org/browse/JDK-8333092

Jaikiran Pai has updated the pull request incrementally with one additional commit since the last revision:

  Alan's suggestion to add a note for SocketImpl.create(boolean stream)

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/19441/files
  - new: https://git.openjdk.org/jdk/pull/19441/files/b382e250..0e3c59f8

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=19441&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=19441&range=00-01

  Stats: 5 lines in 1 file changed: 4 ins; 0 del; 1 mod
  Patch: https://git.openjdk.org/jdk/pull/19441.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/19441/head:pull/19441

PR: https://git.openjdk.org/jdk/pull/19441


More information about the net-dev mailing list