Hi Martin, 

Thanks for explaining the issue to me, that sounds reasonable.

Can I somehow verify that it's the font that has the problem?
Can I fix the font so that the test passes?

Best regards,
  Goetz.


> -----Original Message-----
> From: Martin Balao <mba...@redhat.com>
> Sent: Montag, 13. Mai 2019 18:27
> To: Lindenmaier, Goetz <goetz.lindenma...@sap.com>; 'Severin Gehwolf'
> <sgehw...@redhat.com>; Langer, Christoph <christoph.lan...@sap.com>; jdk-
> updates-...@openjdk.java.net
> Cc: 2d-dev <2d-dev@openjdk.java.net>; build-...@openjdk.java.net; Martin
> Balao Alonso <mbala...@redhat.com>
> Subject: Re: [11u] java/awt/FontMetrics/MaxAdvanceIsMax.java test failure
> (was: [11u] RFR 8210782: Upgrade HarfBuzz to the latest 2.3.1)
> 
> Hi Goetz,
> 
> Thanks for raising this issue.
> 
> I'm not surprised by MaxAdvanceIsMax test failing on some OS. The reason
> is that this test is very OS specific.
> 
> All installed fonts are tested and a static value from each font is used
> for the assertion, after scale calculations. If there is a difference
> between what the font creator set when generating the font file and what
> the OS rendering library returns -considering that different FreeType
> library builds may return different values-, this test will fail.
> 
> The internal test during development phase was more reliable because it
> tested a known-font only. However, shipping a font is not allowed by
> licenses and that's why we changed the test a bit.
> 
> We should probably limit the scope somehow.
> 
> Despite the test failure, I'm still confident that we did the right
> thing in 8218854 [1].
> 
> Kind regards,
> Martin.-
> 
> --
> [1] - https://bugs.openjdk.java.net/browse/JDK-8218854

Reply via email to