RFR: 8235141: Specify the required standard socket options for the socket types in the java.net package

Daniel Fuchs daniel.fuchs at oracle.com
Wed Dec 4 17:14:24 UTC 2019


Doh! My mistake. I dropped this part in the test:
http://cr.openjdk.java.net/~dfuchs/webrev_8235141/webrev.02

cheers,

-- daniel

On 04/12/2019 16:42, Chris Hegarty wrote:
> Daniel,
> 
> On 04/12/2019 14:24, Daniel Fuchs wrote:
>> This test doesn't test NetworkChannel.
> 
> This is what I was referring to:
> 
>    { Configurable.of(SocketChannel.open()), SOCKET_OPTIONS }
> 
>    ..
> 
>     static Configurable<NetworkChannel, IOException> of(NetworkChannel 
> socket) {
>            return new ConfigurableImpl<>(socket, socket::setOption,
>                    socket::getOption, socket::supportedOptions, 
> socket::close);
>         }
> 
> If this is somehow not testing NetworkChannel::supportedOptions, et, 
> then forget my comment. If it is, then it is a candidate to drop, if 
> needed.
> 
>> But it does check that the adapters behave consistently with the spec
>> of their java.net super class.
> 
> Good.
> 
> -Chris.



More information about the net-dev mailing list