RFR [9] 8044773: Refactor jdk.net API so that it can be moved out of the base module

Alan Bateman Alan.Bateman at oracle.com
Tue Apr 26 08:20:27 UTC 2016



On 25/04/2016 22:01, Chris Hegarty wrote:
> One of the remaining open issues in JEP 200 [1] is that the base module
> exports the jdk.net package, thereby violating Principle 4 of JEP 200:
> a Java SE module must not export any non-SE API packages without
> qualification.
>
> http://cr.openjdk.java.net/~chegar/8044773/
> https://bugs.openjdk.java.net/browse/JDK-8044773
>
>
I took a first pass over this and it looks very good.

For the package name then sun.net.extnet repeats "net" and maybe 
sun.net.ext would be better.

For the BaseExtendedSocketOptions then I think the FileDescriptor 
parameter should be the first parameter as that is the file descriptor 
to the socket that the methods operate on.

In ExtendedSocketOptions then I assume the toString is not needed in 
checkValueType.

There are "TODOs" in the tests and I assume it would be better to drop 
those and instead create an issue in JIRA to be addressed once jtreg is 
promoted with support for -limitmods.

There is also a TODO in Sockets.java "check this is working ok" that I 
assume needs attention before this is pushed.

-Alan



More information about the build-dev mailing list