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

Reply via email to