On Wed, 3 Feb 2021 14:48:15 GMT, Alan Bateman <[email protected]> wrote:
>> Patrick Concannon has updated the pull request with a new target base due to
>> a merge or a rebase. The incremental webrev excludes the unrelated changes
>> brought in by the merge/rebase. The pull request contains three additional
>> commits since the last revision:
>>
>> - 8235139: Updated @deprecated text for setSocketFactory methods in Socket
>> and ServerSocket
>> - Merge remote-tracking branch 'origin/master' into JDK-8235139
>> - 8235139: Remove the socket impl factory mechanism
>
> src/java.base/share/classes/java/net/DatagramSocket.java line 959:
>
>> 957: * and using the {@linkplain #DatagramSocket(DatagramSocketImpl)
>> protected
>> 958: * constructor} that takes an {@linkplain DatagramSocketImpl
>> implementation}
>> 959: * as a parameter.
>
> This wording is good, are you planning to add this to the deprecated methods
> in Server and ServerSocket too?
Can do -- see commit e36f91d. I've added similar wording to the equivalent
methods in Server and SeverSocket
-------------
PR: https://git.openjdk.java.net/jdk/pull/2375
