<html>
  <head>
    <meta content="text/html; charset=windows-1252"
      http-equiv="Content-Type">
  </head>
  <body bgcolor="#FFFFFF" text="#000000">
    <div class="moz-cite-prefix">On 2/19/15 8:50 PM, Mandy Chung wrote:<br>
    </div>
    <blockquote cite="mid:54E6BD0C.10704@oracle.com" type="cite">
      <br>
      On 2/19/2015 4:23 PM, Phil Race wrote:
      <br>
      <blockquote type="cite"><a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~prr/8035302/">http://cr.openjdk.java.net/~prr/8035302/</a>
        <br>
      </blockquote>
      <br>
      I'll let Sherman and others to do the detailed review here.  Minor
      comment:
      <br>
      <br>
      FontDescriptor.java
      <br>
         line 56-58: can use try-with-resource.
      <br>
    </blockquote>
    <br>
    Its a <span class="changed">ByteArrayInputStream there is no leak
      potential</span>
    <meta http-equiv="content-type" content="text/html;
      charset=windows-1252">
    <blockquote cite="mid:54E6BD0C.10704@oracle.com" type="cite">   line
      62: should it throw UncheckedIOException?
      <br>
    </blockquote>
    <br>
    Completely theoretical as far as I can tell.<br>
    <br>
    <blockquote cite="mid:54E6BD0C.10704@oracle.com" type="cite">
      <br>
      <blockquote type="cite">
        <br>
        With this change, for the benefit of jigsaw, the Charset support
        needed by the font code
        <br>
        now uses just the public APIs. </blockquote>
      <br>
      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.
      <br>
      <br>
      With this dependency removed,  <top-repo>/modules.xml needs
      to be updated.  Here is the patch:
      <br>
    </blockquote>
    <br>
    I wondered who would do this but if you want it part of this patch,
    no problem I can include it.<br>
    <br>
    -phil.<br>
    <blockquote cite="mid:54E6BD0C.10704@oracle.com" type="cite">
      <br>
      diff --git a/modules.xml b/modules.xml
      <br>
      --- a/modules.xml
      <br>
      +++ b/modules.xml
      <br>
      @@ -294,7 +294,6 @@
      <br>
           </export>
      <br>
           <export>
      <br>
             <name>sun.nio.cs</name>
      <br>
      -      <to>java.desktop</to>
      <br>
             <to>jdk.charsets</to>
      <br>
           </export>
      <br>
           <export>
      <br>
      @@ -602,7 +601,6 @@
      <br>
           <depend>java.prefs</depend>
      <br>
           <depend re-exports="true">java.xml</depend>
      <br>
           <depend
      re-exports="true">java.datatransfer</depend>
      <br>
      -    <depend>jdk.charsets</depend>
      <br>
           <export>
      <br>
             <name>java.applet</name>
      <br>
           </export>
      <br>
      @@ -1481,10 +1479,6 @@
      <br>
         <module>
      <br>
           <name>jdk.charsets</name>
      <br>
           <depend>java.base</depend>
      <br>
      -    <export>
      <br>
      -      <name>sun.nio.cs.ext</name>
      <br>
      -      <to>java.desktop</to>
      <br>
      -    </export>
      <br>
         </module>
      <br>
         <module>
      <br>
           <name>jdk.compiler</name>
      <br>
      <br>
      Run make verify-modules to verify the dependencis or make images
      will do it too.
      <br>
      <br>
      Mandy
      <br>
      [1] <a class="moz-txt-link-freetext" href="http://openjdk.java.net/jeps/200">http://openjdk.java.net/jeps/200</a>
      <br>
      <br>
    </blockquote>
    <br>
  </body>
</html>