<AWT Dev> RFR(XS): 8155156: Remove remaining sun.misc.* imports from the jdk repo

Chris Hegarty chris.hegarty at oracle.com
Tue Apr 26 17:42:20 UTC 2016


On 26 Apr 2016, at 18:33, Volker Simonis <volker.simonis at gmail.com> wrote:

> Hi,
> 
> can I please have a review for this trivial change:
> 
> http://cr.openjdk.java.net/~simonis/webrevs/2016/8155156/
> https://bugs.openjdk.java.net/browse/JDK-8155156

Thank you Volker.  Reviewed.

-Chris.

> The fix for "8153737: Unsupported Module" moved sun.misc to the
> jdk.unsupported module and removed sun.misc.* imports.
> 
> Unfortunately, it forgot to remove the sun.misc.* imports from:
> 
> src/java.base/share/classes/sun/nio/ch/AbstractPollSelectorImpl.java
> src/java.base/unix/classes/sun/nio/ch/PollSelectorImpl.java
> 
> which are only used on AIX. Because the base module doesn't require
> jdk.unsupported, this leads to a build error on AIX.
> 
> The last remaining sun.misc.* import in
> 
> src/java.desktop/unix/classes/sun/awt/X11/XToolkit.java
> 
> isn't required any more as well after "8147544: Remove
> sun.misc.ManagedLocalsThread from java.desktop" has been pushed. It
> doesn't lead to a build error because the java.desktop module still
> requires jdk.unsupported, but as the import isn't necessary any more,
> I think it's better to remove it as well.
> 
> I build and smoke tested on Linux, Solaris and AIX.
> 
> Thank you and best regards,
> Volker



More information about the awt-dev mailing list