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

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


On Sat, 6 Apr 2024 09:10:10 GMT, Alan Bateman <alanb at openjdk.org> wrote:

>> Jaikiran Pai has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   remove some @see from constructors
>
> src/java.base/share/classes/java/net/ServerSocket.java line 189:
> 
>> 187:      * factory, that {@linkplain SocketImplFactory#createSocketImpl()
>> 188:      * factory's createSocketImpl method} is called to create
>> 189:      * the actual socket implementation. Otherwise a system-default
> 
> I think this would be better if you link  "server socket implementation factory" to SocketImplFactory, and link "createSocketImpl" to the createSocketImpl method.

Done - I've updated the PR with these changes to both ServerSocket and Socket class.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/18646#discussion_r1554793874


More information about the net-dev mailing list