Great , thanks for the review ! Do I need a second one or is this considered simple/trivial enought ?
Best regards, Matthias From: Jayathirth D v <[email protected]> Sent: Mittwoch, 1. Juli 2020 10:04 To: Baesken, Matthias <[email protected]> Cc: [email protected]; Phil Race ([email protected]) <[email protected]> Subject: Re: [OpenJDK 2D-Dev] RFR [XS]: 8248468: java/awt/font/DefaultFontTest/DefaultFontTest.java fails in SunFontManager.findFont2D Looks good to me. Thanks, Jay On 01-Jul-2020, at 12:10 PM, Baesken, Matthias <[email protected]<mailto:[email protected]>> wrote: Ping … would be nice to get a review … From: Baesken, Matthias Sent: Montag, 29. Juni 2020 10:06 To: [email protected]<mailto:[email protected]> Subject: RFR [XS]: 8248468: java/awt/font/DefaultFontTest/DefaultFontTest.java fails in SunFontManager.findFont2D Hello, please review this small fix. It was noticed that the jtreg test java/awt/font/DefaultFontTest/DefaultFontTest.java fails on one of our RedHat 7.1 machines with this exception : java.lang.NullPointerException: Cannot invoke "String.toLowerCase(java.util.Locale)" because "name" is null at java.desktop/sun.font.SunFontManager.findFont2D(SunFontManager.java:1891) at java.desktop/sun.font.SunFontManager.getDefaultPhysicalFont(SunFontManager.java:1036) at DefaultFontTest.main(DefaultFontTest.java:57) at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method) at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:64) at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) at java.base/java.lang.reflect.Method.invoke(Method.java:564) at com.sun.javatest.regtest.agent.MainWrapper$MainThread.run(MainWrapper.java:127) at java.base/java.lang.Thread.run(Thread.java:832) Looks like name.toLowerCase(Locale.ENGLISH); is called on a font name that is null, this should be checked / avoided . Bug/webrev : https://bugs.openjdk.java.net/browse/JDK-8248468 http://cr.openjdk.java.net/~mbaesken/webrevs/8248468.0/ Thanks, Matthias
