<AWT Dev>  Review request for 8154539 Examine the desktop module's use of sun.misc.SoftCache
alexandr.scherbatiy at oracle.com
Wed May 11 21:43:36 UTC 2016
Could you review the updated fix:
- The test/com/sun/jdi/ReferrersTest.java test is updated to use
- The sun.misc.GThreadHelper class is moved to the sun.awt package in
the java.desktop module
The jdk.unsupported dependency was removed from the
The sun.misc.GThreadHelper class is called by JNI and may be it does
not require the declared dependency to the jdk.unsupported module. In
any case it seems better to move the GThreadHelper class to the
On 12/05/16 00:15, Mandy Chung wrote:
>> On May 11, 2016, at 1:07 PM, Alexander Scherbatiy <alexandr.scherbatiy at oracle.com> wrote:
>> Could you review the fix:
>> bug: https://bugs.openjdk.java.net/browse/JDK-8154539
>> webrev: http://cr.openjdk.java.net/~alexsch/8154539/webrev.00
>> This is a request from the jigsaw team to get rid of the dependency to jdk.unsupported module from the java.desktop module.
>> The fix moves the sun.misc.SoftCache class to the sun.awt package in the java.desktop module.
> Looks okay. Moving sun.misc.SoftCache to java.desktop is fine.
> test/com/sun/jdi/ReferrersTest.java references sun.misc.SoftCache. I don’t know whether this test finds sun.misc.SoftCache in the reference graph at all. I suggest to update the test to check “sun.awt.SoftCache” instead and file a bug to the serviceability to follow up if that check is needed or not.
More information about the awt-dev