RFR 8224477 [13] java.net.Socket::setOption - implementation mismatch to spec

Alan Bateman Alan.Bateman at oracle.com
Mon May 27 16:11:38 UTC 2019


On 27/05/2019 16:48, Chris Hegarty wrote:
>
>
> Updated webrev:
>   https://cr.openjdk.java.net/~chegar/8224477/webrev.04/
This version looks good.

A minor nit is that there's a blank line added to 
SocketChannelImpl.setOption that separates a comment from the code that 
it documents, I assume this is left over from previous iteration when 
there was a null check in the SO_REUSEADDR handling.

Also the two TestDefaultBehavior tests both name the test impl 
"DelegatingSocketImpl" but it doesn't really delegate, they are really 
no-op impls.

On the CSR. The Problem section suggests that supportedOptions returned 
a set of options that are reasonable for a ServerSocket. I assume this 
should be client Socket as most of the options were not interesting for 
a ServerSocket.

-Alan






More information about the net-dev mailing list