RFR 8148609: supportedOptions() methods return a mutable set

Alan Bateman Alan.Bateman at oracle.com
Wed Mar 2 08:19:03 UTC 2016



On 02/03/2016 06:47, vyom wrote:
> Hi Chris/Alan,
>
> Thanks for review, please find the updated 
> webrev(http://cr.openjdk.java.net/~nkumar/vyom/8148609/webrev0.1/index.html 
> <http://cr.openjdk.java.net/%7Enkumar/vyom/8148609/webrev0.1/index.html>). 
>
This looks much better.

I see the test is named SupportedOptionTest and so hints that it tests 
more than it does. I wonder if we could find a better name for it, maybe 
ImmutableOptions or something along those lines.

-Alan


More information about the net-dev mailing list