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