Looks good to me. Thanks, Jay
> On 06-Jul-2020, at 12:43 PM, Baesken, Matthias <matthias.baes...@sap.com> > wrote: > > Hello, please review this small change to font related logging . > > We have a lot of font logging calls in java.desktop that look similar to this > coding : > > if (FontUtilities.isLogging()) { > FontUtilities.getLogger().info("Here comes my important info"); > } > > This coding could be simplified by adding static log methods to > FontUtilities.java > > public static void logWarning(String s); > public static void logInfo(String s); > public static void logSevere(String s); > > doing the isLogging check + FontUtilities.getLogger(). … > > > > Bug/webrev : > > https://bugs.openjdk.java.net/browse/JDK-8248802 > <https://bugs.openjdk.java.net/browse/JDK-8248802> > > http://cr.openjdk.java.net/~mbaesken/webrevs/8248802.0/ > <http://cr.openjdk.java.net/~mbaesken/webrevs/8248802.0/> > > > Thanks, Matthias