You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@any23.apache.org by ha...@apache.org on 2018/10/29 17:00:45 UTC

[4/4] any23 git commit: ANY23-38 remove references to JUL

ANY23-38 remove references to JUL


Project: http://git-wip-us.apache.org/repos/asf/any23/repo
Commit: http://git-wip-us.apache.org/repos/asf/any23/commit/f87ac66b
Tree: http://git-wip-us.apache.org/repos/asf/any23/tree/f87ac66b
Diff: http://git-wip-us.apache.org/repos/asf/any23/diff/f87ac66b

Branch: refs/heads/master
Commit: f87ac66bc6d96a580468d1cbf9b963365662fb1f
Parents: 5aba30b
Author: Hans <fi...@gmail.com>
Authored: Mon Oct 29 11:45:26 2018 -0500
Committer: Hans <fi...@gmail.com>
Committed: Mon Oct 29 11:45:26 2018 -0500

----------------------------------------------------------------------
 .../main/java/org/apache/any23/cli/ToolRunner.java |  7 -------
 .../main/java/org/apache/any23/util/LogUtils.java  | 17 ++++++++++-------
 2 files changed, 10 insertions(+), 14 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/any23/blob/f87ac66b/cli/src/main/java/org/apache/any23/cli/ToolRunner.java
----------------------------------------------------------------------
diff --git a/cli/src/main/java/org/apache/any23/cli/ToolRunner.java b/cli/src/main/java/org/apache/any23/cli/ToolRunner.java
index a19ecf1..39e0c0a 100644
--- a/cli/src/main/java/org/apache/any23/cli/ToolRunner.java
+++ b/cli/src/main/java/org/apache/any23/cli/ToolRunner.java
@@ -22,7 +22,6 @@ import com.beust.jcommander.Parameter;
 import com.beust.jcommander.converters.FileConverter;
 import org.apache.any23.Any23;
 import org.apache.any23.plugin.Any23PluginManager;
-import org.apache.any23.util.LogUtils;
 
 import java.io.File;
 import java.io.IOException;
@@ -118,12 +117,6 @@ public final class ToolRunner {
             return 1;
         }
 
-        if (verbose) {
-            LogUtils.setVerboseLogging();
-        } else {
-            LogUtils.setDefaultLogging();
-        }
-
         long start = currentTimeMillis();
         int exit = 0;
 

http://git-wip-us.apache.org/repos/asf/any23/blob/f87ac66b/core/src/main/java/org/apache/any23/util/LogUtils.java
----------------------------------------------------------------------
diff --git a/core/src/main/java/org/apache/any23/util/LogUtils.java b/core/src/main/java/org/apache/any23/util/LogUtils.java
index 30b24ca..727df55 100644
--- a/core/src/main/java/org/apache/any23/util/LogUtils.java
+++ b/core/src/main/java/org/apache/any23/util/LogUtils.java
@@ -17,23 +17,26 @@
 
 package org.apache.any23.util;
 
-import java.util.logging.Level;
-import java.util.logging.Logger;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
+
+import java.lang.invoke.MethodHandles;
 
 /**
  * Provides specific application logger configuration methods.
+ * @deprecated since 2.3. JUL is no longer used in this project.
  */
+@Deprecated
 public class LogUtils {
 
+    private static final Logger logger = LoggerFactory.getLogger(MethodHandles.lookup().lookupClass());
+
     public static void setDefaultLogging() {
-        Logger.getLogger("").setLevel(Level.WARNING);
-        Logger.getLogger("").getHandlers()[0].setLevel(Level.ALL);
+        logger.error("setDefaultLogging() is not supported.");
     }
 
     public static void setVerboseLogging() {
-        Logger.getLogger("").setLevel(Level.INFO);
-        Logger.getLogger("org.apache.any23").setLevel(Level.ALL);
-        Logger.getLogger("").getHandlers()[0].setLevel(Level.ALL);
+        logger.error("setVerboseLogging() is not supported.");
     }
 
 }