RFR[8234103]: DatagramSocketImpl::socket is not needed
Daniel Fuchs
daniel.fuchs at oracle.com
Thu Nov 14 12:12:45 UTC 2019
Hi Patrick,
Thanks for helping getting rid of this old technical debt!
In DualStackPlainDatagramSocketImpl.java:
64 DualStackPlainDatagramSocketImpl(boolean exclBind) {
65 super(false);
Maybe add a comment to clarify the meaning of `false`:
65 super(false); // DualStackPlainDatagramSocketImpl doesn't
support multicast
Otherwise looks good!
best regards,
-- daniel
On 14/11/2019 11:58, Patrick Concannon wrote:
> Hi,
>
> Could someone please review my fix for issue JDK-8234103
> 'DatagramSocketImpl::socket is not needed' ?
>
> DatagramSocketImpl has a socket field that links back to the
> DatagramSocket, which is only used to determine whether this
> DatagramSocket support Multicast or not. This fix removes the
> DatagramSocket object and replaces it with a boolean 'isMulticast' field
> in AbstractPlainDatagramSocketImpl.
>
> bug: https://bugs.openjdk.java.net/browse/JDK-8234103
> webrev: http://cr.openjdk.java.net/~pconcannon/8234103/webrevs/webrev.00/
>
>
> Kind regards,
>
> Patrick
>
More information about the net-dev
mailing list