RFR[8234103]: DatagramSocketImpl::socket is not needed

Patrick Concannon patrick.concannon at oracle.com
Thu Nov 14 11:58:00 UTC 2019


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