RFR 8216417: cleanup of IPv6 scope-id handling

Alan Bateman Alan.Bateman at oracle.com
Tue Jun 11 12:19:08 UTC 2019


On 11/06/2019 12:11, Michael McMahon wrote:
> Thanks for the review Alan.
> I've made the changes suggested by you and Daniel
> and added a test to Scoping.java for checking the connected IP address.
>
> http://cr.openjdk.java.net/~michaelm/8216417/webrev.3/index.html
One suggestion for the AbstractPlainSocketImpl connect methods is to set 
this.address and this.port together and include a comment to say that 
the socket address is recorded before attempting to connect.

A few formatting nits to check: AbstractPlainDatagramSocketImpl L134 and 
the spurious blank line at IPAddressUtil L355.

In the PromiscuousIPv6 test then I'm wondering if the check for the 
3.10.0.* kernel is needed or not. I suspect this may have been in 
Pavel's original changes. Maybe Pavel has the history on that?

I see the Scoping test has been expanded. In datagramTest you can use 
try-with-resources. Also shouldn't there be an equivalent for 
DatagramChannel.

Everything else looks good.

-Alan


More information about the nio-dev mailing list