Merge pull request #140 from jamezp/LOGMGR-175

[LOGMGR-175] Revert the change to use the StandardOutputStreams on th…
jboss-logging · Sep 21, 2017 · aa9959c · aa9959c
2 parents 76734a3 + f4ae66a
commit aa9959c
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 6 deletions.
diff --git a/src/main/java/org/jboss/logmanager/StandardOutputStreams.java b/src/main/java/org/jboss/logmanager/StandardOutputStreams.java
@@ -26,7 +26,7 @@
  *
  * @author <a href="mailto:jperkins@redhat.com">James R. Perkins</a>
  */
-@SuppressWarnings("WeakerAccess")
+@SuppressWarnings({"WeakerAccess", "unused"})
 public class StandardOutputStreams {
     public static final PrintStream stdout = System.out;
     public static final PrintStream stderr = System.err;

diff --git a/src/main/java/org/jboss/logmanager/handlers/ConsoleHandler.java b/src/main/java/org/jboss/logmanager/handlers/ConsoleHandler.java
@@ -26,13 +26,14 @@
 
 import java.util.logging.Formatter;
 
-import org.jboss.logmanager.StandardOutputStreams;
 import org.jboss.logmanager.formatters.Formatters;
 
 /**
  * A console handler which writes to {@code System.out} by default.
  */
 public class ConsoleHandler extends OutputStreamHandler {
+    private static final OutputStream out = System.out;
+    private static final OutputStream err = System.err;
 
     /**
      * The target stream type.
@@ -94,8 +95,8 @@ public ConsoleHandler(final Target target) {
     public ConsoleHandler(final Target target, final Formatter formatter) {
         super(formatter);
         switch (target) {
-            case SYSTEM_OUT: setOutputStream(wrap(StandardOutputStreams.stdout)); break;
-            case SYSTEM_ERR: setOutputStream(wrap(StandardOutputStreams.stderr)); break;
+            case SYSTEM_OUT: setOutputStream(wrap(out)); break;
+            case SYSTEM_ERR: setOutputStream(wrap(err)); break;
             case CONSOLE: setWriter(wrap(console)); break;
             default: throw new IllegalArgumentException();
         }
@@ -108,8 +109,8 @@ public ConsoleHandler(final Target target, final Formatter formatter) {
      */
     public void setTarget(Target target) {
         switch (target) {
-            case SYSTEM_OUT: setOutputStream(wrap(StandardOutputStreams.stdout)); break;
-            case SYSTEM_ERR: setOutputStream(wrap(StandardOutputStreams.stderr)); break;
+            case SYSTEM_OUT: setOutputStream(wrap(out)); break;
+            case SYSTEM_ERR: setOutputStream(wrap(err)); break;
             case CONSOLE: setWriter(wrap(console)); break;
             default: throw new IllegalArgumentException();
         }