[OpenJDK 2D-Dev] RFR: 8035302: Eliminate dependency on sun.nio.cs from AWT and Motif code

[Mandy Chung]

On 2/19/2015 4:23 PM, Phil Race wrote:
> http://cr.openjdk.java.net/~prr/8035302/

I'll let Sherman and others to do the detailed review here.  Minor comment:

FontDescriptor.java
    line 56-58: can use try-with-resource.
    line 62: should it throw UncheckedIOException?

>
> With this change, for the benefit of jigsaw, the Charset support 
> needed by the font code
> now uses just the public APIs. 

java.desktop to the internal of charsets is an undesirable edge in the 
module graph [1].  Thank you for eliminating it.  It's not ideal to copy 
sun.nio code while some day these sun.awt.motif.** will no longer be 
needed and can be removed.

With this dependency removed,  <top-repo>/modules.xml needs to be 
updated.  Here is the patch:

diff --git a/modules.xml b/modules.xml
--- a/modules.xml
+++ b/modules.xml
@@ -294,7 +294,6 @@
      </export>
      <export>
        <name>sun.nio.cs</name>
-      <to>java.desktop</to>
        <to>jdk.charsets</to>
      </export>
      <export>
@@ -602,7 +601,6 @@
      <depend>java.prefs</depend>
      <depend re-exports="true">java.xml</depend>
      <depend re-exports="true">java.datatransfer</depend>
-    <depend>jdk.charsets</depend>
      <export>
        <name>java.applet</name>
      </export>
@@ -1481,10 +1479,6 @@
    <module>
      <name>jdk.charsets</name>
      <depend>java.base</depend>
-    <export>
-      <name>sun.nio.cs.ext</name>
-      <to>java.desktop</to>
-    </export>
    </module>
    <module>
      <name>jdk.compiler</name>

Run make verify-modules to verify the dependencis or make images will do it too.

Mandy
[1] http://openjdk.java.net/jeps/200