Hi,

Could someone please review my changes for the removal of the legacy 
`PlainSocketImpl` and `PlainDatagramSocketImpl` implementations?

In JDK 13, JEP 353 provided a drop in replacement for the legacy 
`PlainSocketImpl` implementation. Since JDK 13, the `PlainSocketImpl` 
implementation was no longer used but included a mitigation mechanism to reduce 
compatibility risks in the form of a JDK-specific property 
`jdk.net.usePlainSocketImpl` allowing to switch back to the old implementation. 
Similarly, in JDK 15, JEP 373 provided a new implementation for 
`DatagramSocket` and `MulticastSocket`, with a JDK-specific property 
`jdk.net.usePlainDatagramSocketImpl` also allowing the user to switch back to 
the old implementation in case of compatibility issue.

As these implementations (and the mechanisms they use to enable them to 
mitigate compatibility issues) have been deemed no longer necessary, they now 
represent a maintenance burden. This patch looks at removing them from the JDK.

Kind regards,
Patrick

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

Commit messages:
 - 8253119: Remove the legacy PlainSocketImpl and PlainDatagramSocketImpl 
implementation

Changes: https://git.openjdk.java.net/jdk/pull/4574/files
 Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=4574&range=00
  Issue: https://bugs.openjdk.java.net/browse/JDK-8253119
  Stats: 11046 lines in 78 files changed: 137 ins; 10836 del; 73 mod
  Patch: https://git.openjdk.java.net/jdk/pull/4574.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/4574/head:pull/4574

PR: https://git.openjdk.java.net/jdk/pull/4574

Reply via email to