[OpenJDK 2D-Dev] RFC: Fix for JDK-8188030

[Mario Torre]

Hi Phil,

I think I may be fine if we need to live with a local patch in our RPM in
8, this use case should be pretty rare, although I'll check first if
there’s some ifdefs that I can use for the backport to 8.

I tried to figure out if I could just change the logic but all I could get
was NPEs.

Perhaps a more fine tuned search to exclude CFF in the first place, but I’m
not sure how to craft the query for FontConfig in this case.

I’ll push to 10 for now and backport to 9 in the meantime. Do I need
another reviewer ok?

Cheers,
Mario

On Wed 27. Sep 2017 at 23:15, Phil Race <philip.race at oracle.com> wrote:

> This is fine for JDK 9 + 10 but the reason CFF fonts are discarded
> is that we weren't supporting them properly in Oracle JDK until 9.
> We should have removed this check in 9 but it was forgotten.
>
> If you backport this to 7 and 8 it will be a problem there - for
> Oracle JDK, not OpenJDK.
>
> Although it won't matter for openjdk7 .. since Oracle isn't using that
> forest any more
>
> But it will be an issue for 8. Not sure how to handle that but
> it should not be backported without a resolution there.
>
> -phil.
>
> On 09/27/2017 06:52 AM, Mario Torre wrote:
> > Hi all,
> >
> > I would like to propose a fix for
> > https://bugs.openjdk.java.net/browse/JDK-8188030.
> >
> > The issue is basically that CFF fonts are considered better match than
> > Type 1, but are discarded, leaving the font array with no elements.
> >
> > The fix is here:
> >
> > http://cr.openjdk.java.net/~neugens/8188030/webrev.01/
> >
> > I attached a reproducer to the bug report, but you need a somewhat
> > minimal system setup for that to work.
> >
> > The fix is for OpenJDK 10 at this point, but I plan to backport it all
> > the way down to 7.
> >
> > Any comments?
> >
> > Cheers,
> > Mario
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/2d-dev/attachments/20170928/0c497e71/attachment.html>