<html>
  <head>
    <meta content="text/html; charset=utf-8" http-equiv="Content-Type">
  </head>
  <body bgcolor="#FFFFFF" text="#000000">
    <p>Hi Phil,<br>
    </p>
    Looks good to me . One thing is that, the check for<span
      class="changed"> platformID != MAC_PLATFORM_ID</span><br>
    shouldn't we do the same in initAllNames()?<br>
    <br>
    Also, in the test, I guess we do not need <br>
    <pre>33 import java.io.BufferedInputStream;
  35 import java.io.FileInputStream;
  36 import java.io.InputStream;
  37 </pre>
    and copyright should be 2017.<br>
    <br>
    Regards<br>
    Prasanta<br>
    <div class="moz-cite-prefix">On 1/5/2017 5:25 AM, Philip Race wrote:<br>
    </div>
    <blockquote cite="mid:586D8B8C.4010600@oracle.com" type="cite">
      <br>
      Bug : <a class="moz-txt-link-freetext" href="https://bugs.openjdk.java.net/browse/JDK-6232267">https://bugs.openjdk.java.net/browse/JDK-6232267</a>
      <br>
      Webrev : <a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~prr/6232267/">http://cr.openjdk.java.net/~prr/6232267/</a>
      <br>
      <br>
      Fonts such as "Menlo" on OS X have no Microsoft names
      <br>
      (see <a class="moz-txt-link-freetext" href="https://www.microsoft.com/typography/otspec/name.htm">https://www.microsoft.com/typography/otspec/name.htm</a>)
      <br>
      so historically we rejected these.
      <br>
      <br>
      Now that we support OS X, this is more of an issue, and
      <br>
      it is possible to run into a case where we fail to parse these
      fonts
      <br>
      when they go through the Font.createFont() path, even on OS X.
      <br>
      <br>
      This adds that basic support for parsing Apple names so that
      <br>
      we load the font. This support already exists on JavaFX.
      <br>
      <br>
      -phil.
      <br>
      <br>
    </blockquote>
    <br>
  </body>
</html>