RFR [9] 8153737: Unsupported Module

Chris Hegarty chris.hegarty at oracle.com
Fri Apr 8 15:35:23 UTC 2016

On 08/04/16 03:52, Mandy Chung wrote:
>> On Apr 7, 2016, at 10:14 AM, Chris Hegarty <chris.hegarty at oracle.com> wrote:
>> Enough technical debt has been paid down that we can now create the new
>> JDK-specific module as proposed by JEP 260 [1], named jdk.unsupported.
>> This module will initially contain, and export, the sun.misc package,
>> and will eventually export the sun.reflect package too ( once it has
>> been sanitized ). The jdk.unsupported module will be present in full JRE
>> and JDK images.
>> http://cr.openjdk.java.net/~chegar/8153737/
>> https://bugs.openjdk.java.net/browse/JDK-8153737
> I’m glad to see jdk.unsupported module be created.
> jdeps tests are intended to test references to sun.misc and flagged as internal API. The jdeps test changes should be reverted.
> It’s exported to allow existing applications to continue to run on JDK 9 while sun.misc and other critical internal APIs remains to be internal APIs.  jdeps -jdkinternals should flag them and get developers’ attention to prepare to migrate when the replacements are available.

I moved the tests from a directory named 'jdk.unsupported' to
unsupported', as other tests, in test/tools/jdeps/module, use
test/tools/jdeps as a test library, and the directory/test lib
name is conflicting with the module name. jtreg fails immediately.
The updates I made check that jdeps identifies both jdk.internal.misc
and sun.misc Unsafe as internal APIs.

> Other than that, looks good.

Thanks for the Review.

> There are so many tests that have @modules java.base sun/misc but unused.  It’s good that you clean that up.   I agree with Alan that JDK modules depending on jdk.unsupported should get the priority to get eliminated by FC.

I bumped the priority, and target the issues to 9.


More information about the core-libs-dev mailing list