<html>
  <head>
    <meta content="text/html; charset=UTF-8" http-equiv="Content-Type">
  </head>
  <body bgcolor="#FFFFFF" text="#000000">
    Oh two other things.<br>
    <br>
    1) The change should be pushed to jdk/client not jdk/jdk<br>
    2) If you are making changes anyway maybe enumerate the imports
    rather than wild-carding..<br>
    <br>
    -phil.<br>
    <br>
    On 2/21/19, 5:47 PM, Philip Race wrote:
    <blockquote cite="mid:5C6F54A7.80708@oracle.com" type="cite">
      <meta content="text/html; charset=UTF-8" http-equiv="Content-Type">
      I submitted a test job to see what it did on our test platforms :
      Solaris, WIndows, Mac as well as Linux<br>
      and I got lot of failures on Solaris .. 38 of them .. a sample
      below but not just for obscure fonts,<br>
      it included Arial (in various styles), Arial Black, Arial Narrow
      ... <br>
      <br>
      They are all at char width 1 or 2 .. since Arial is also available
      on Mac + Windows and didn't<br>
      fail there I wonder if it is due to the freetype installed on
      Solaris ?<br>
      <br>
      Not sure what to do ... exclude the smallest sizes for now ?<br>
      <br>
      -phil.<br>
      <br>
      <meta http-equiv="content-type" content="text/html; charset=UTF-8">
      <pre>Testing java.awt.Font[family=AR PL ShanHeiSun Uni,name=AR PL ShanHeiSun Uni,style=bolditalic,size=1] in FT_LOAD_TARGET_MONO
getMaxAdvance: 2
Max char width: 1
PASSED
.........................
Testing java.awt.Font[family=AlArabiya,name=AlArabiya,style=bolditalic,size=1] in FT_LOAD_TARGET_MONO
getMaxAdvance: 2
Max char width: 1
PASSED
.........................
Testing java.awt.Font[family=AlBattar,name=AlBattar,style=bolditalic,size=1] in FT_LOAD_TARGET_MONO
getMaxAdvance: 2
Max char width: 1
PASSED
.........................
Testing java.awt.Font[family=AlHor,name=AlHor,style=bolditalic,size=1] in FT_LOAD_TARGET_MONO
getMaxAdvance: 2
Max char width: 1
PASSED
.........................
Testing java.awt.Font[family=AlManzomah,name=AlManzomah,style=bolditalic,size=1] in FT_LOAD_TARGET_MONO
getMaxAdvance: 2
Max char width: 1
PASSED
....


</pre>
      <br>
      <br>
      On 2/21/19, 2:19 PM, Sergey Bylokhov wrote:
      <blockquote
        cite="mid:539930ae-017a-3e92-de83-db80e73656dc@oracle.com"
        type="cite">Hi, Martin. <br>
        <br>
        What exceptions do you expect below? I think any exception will
        means a bug in jdk, no? <br>
        <br>
                try { <br>
                    GraphicsEnvironment e = <br>
                       
        GraphicsEnvironment.getLocalGraphicsEnvironment(); <br>
                    fonts = e.getAllFonts(); <br>
                } catch (Exception e) {} <br>
        <br>
        Even if you skip an Exception, you will get NPE later at line
        "103 for (Font f : fonts) {" <br>
        <br>
        BTW probably the test can be moved to the
        "java/awt/FontMetrics/" folder. <br>
        <br>
        On 21/02/2019 13:27, Martin Balao wrote: <br>
        <blockquote type="cite">Hi Phil, <br>
          <br>
          On 2/19/19 10:34 PM, Philip Race wrote: <br>
          <blockquote type="cite">One more thing about the test, I am
            not sure why you need to use <br>
            sun.font.FontDesignMetrics directly ? <br>
            <br>
            Isn't it enough to create a BufferedImage and get an
            appropriate <br>
            FRC and FM by setting the properties on the graphics for
            that ? <br>
            <br>
          </blockquote>
          <br>
          Yes, that's better, there's no need to use an internal API
          here. <br>
          <br>
          Webrev.02: <br>
          <br>
            * <a moz-do-not-send="true" class="moz-txt-link-freetext"
href="http://cr.openjdk.java.net/%7Embalao/webrevs/8218854/8218854.webrev.02/">http://cr.openjdk.java.net/~mbalao/webrevs/8218854/8218854.webrev.02/</a>
          <br>
            * <a moz-do-not-send="true" class="moz-txt-link-freetext"
href="http://cr.openjdk.java.net/%7Embalao/webrevs/8218854/8218854.webrev.02.zip">http://cr.openjdk.java.net/~mbalao/webrevs/8218854/8218854.webrev.02.zip</a>
          <br>
          <br>
          Are we good to go with Webrev.02? <br>
          <br>
          Thanks, <br>
          Martin.- <br>
          <br>
        </blockquote>
        <br>
        <br>
      </blockquote>
    </blockquote>
  </body>
</html>