On 07/25/2016 07:14 AM, Martin Walsh wrote:
On 07/22/16 05:59 PM, Phil Race wrote:
Hi,
That looks reasonable .. but before an OK I would like to see the
results of running the test case in
the bug report after that fix. Run the test case with
-Dsun.java2d.debugfonts=true so we can see
what the JDK is doing internally. You can add the log to the bug report.
Thanks, I have attached the output of this to the bug.
I took a look through these and "no surprises", so the fix seems fine
(approved).
Do you need me to push this for you ?
Basically fontconfig is also likely to be subsequently invoked to build
the list of real fonts underlying
the logical font names (eg JDK sansserif which is the equivalent of
fontconfig's sans).
I suspect they will all end up just falling back to the Lucida Sans
Regular font shipped with JDK.
But for OpenJDK that won't be there .. so you may also want to see what
happens if you
then delete (move elsewhere) that font from jdk/lib/fonts. Maybe not a
VM crash but a Java NPE perhaps
as no fonts at all is not a supportable configuration.
Yes, you get a NPE.
# java Family
Exception in thread "main" java.lang.NullPointerException
...
Yes, this is what I expected there. Perhaps we should be throwing some
other kind
of exception here .. if there really are no fonts anywhere .. but that
is beyond the
immediate scope of this fix.
-phil.
