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

Patrick Concannon patrick.concannon at oracle.com
Thu Nov 14 12:24:39 UTC 2019


Hi Daniel,


I've added the comment, and you can find it in the new webrev below.

webrev: http://cr.openjdk.java.net/~pconcannon/8234103/webrevs/webrev.01/


Kind regards,

Patrick


On 14/11/2019 12:12, Daniel Fuchs wrote:
> 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