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