RFR: 8329745: Update the documentation of ServerSocket and Socket to refer to StandardSocketOptions instead of legacy SocketOptions [v4]

Jaikiran Pai jpai at openjdk.org
Sun Apr 7 01:43:31 UTC 2024


> Can I please get a review of this doc-only changes to java.net.ServerSocket and java.net.Socket classes?
> 
> As noted in https://bugs.openjdk.org/browse/JDK-8329745, these classes currently refer to the legacy `java.net.SocketOptions` interface and instead should be refering to the newer `java.net.StandardSocketOptions` class. The commit in this PR updates such references. This change intentionally doesn't do any code changes to use the `StandardSocketOptions` class - that can be done separately if desired at a later point (after testing for any compatibility issues). Finally, there are a few places in ServerSocket and Socket documentation which will continue to refer to java.net.SocketOptions legacy interface because few of the options aren't available in StandardSocketOptions class (for example, `SO_TIMEOUT`).
> 
> I ran `make docs-image` locally with this change and the generated doc looks OK to me.

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

  separate links for SocketImplFactory and createSocketImpl

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/18646/files
  - new: https://git.openjdk.org/jdk/pull/18646/files/34f919b5..70fc7fc5

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=18646&range=03
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=18646&range=02-03

  Stats: 33 lines in 2 files changed: 0 ins; 0 del; 33 mod
  Patch: https://git.openjdk.org/jdk/pull/18646.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/18646/head:pull/18646

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


More information about the net-dev mailing list