From 71a9abdad997844546404c755be48ccc80a752f8 Mon Sep 17 00:00:00 2001 From: games647 Date: Wed, 1 Jun 2016 18:28:22 +0200 Subject: [PATCH] Remove redundant isLoggable check. It's always level ALL Looks like the java logger does not reflect the log4j level settings :( --- src/main/java/fr/xephi/authme/ConsoleLogger.java | 10 ++++------ 1 file changed, 4 insertions(+), 6 deletions(-) diff --git a/src/main/java/fr/xephi/authme/ConsoleLogger.java b/src/main/java/fr/xephi/authme/ConsoleLogger.java index fc208742..ef9561a9 100644 --- a/src/main/java/fr/xephi/authme/ConsoleLogger.java +++ b/src/main/java/fr/xephi/authme/ConsoleLogger.java @@ -69,12 +69,10 @@ public final class ConsoleLogger { public static void debug(String message) { if (enableDebug) { - if (logger.isLoggable(Level.FINE)) { - //creating and filling an exception is a expensive call - logger.log(Level.FINE, message + ' ' + Thread.currentThread().getName(), new Exception()); - } else { - logger.log(Level.FINE, "{0} {1}", new Object[]{message, Thread.currentThread().getName()}); - } + //creating and filling an exception is a expensive call + //->so it should be removed as soon #418 is fixed + //logger.isLoggable does not work because the plugin logger is always ALL + logger.log(Level.FINE, message + ' ' + Thread.currentThread().getName(), new Exception()); if (useLogging) { writeLog("Debug: " + Thread.currentThread().getName() + ':' + message);