8213210: Change ServerSocket(SocketImpl impl) constructor to protected access

Alan Bateman Alan.Bateman at oracle.com
Wed Oct 31 19:32:45 UTC 2018


On 31/10/2018 19:26, Brian Burkhalter wrote:
>
>> On Oct 31, 2018, at 12:20 PM, Brian Burkhalter 
>> <brian.burkhalter at oracle.com <mailto:brian.burkhalter at oracle.com>> wrote:
>>
>> -   ServerSocket(SocketImpl impl) {
>> +   protected ServerSocket(SocketImpl impl) {
>>       this.impl = impl;
>>       impl.setServerSocket(this); // <- NPE if impl == null
>>   }
>
> Oops, it can throw null as indicated above. Revised patch below.
>
Good, I think looks okay.

-Alan
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/net-dev/attachments/20181031/0392aebd/attachment.html>


More information about the net-dev mailing list