svn commit: r824543 - in /tuscany/java/sca/modules/host-jetty/src/main/java/org/apache/tuscany/sca/http/jetty: JettyLogger.java JettyServer.java

[rf...@apache.org]

Author: rfeng
Date: Mon Oct 12 23:47:12 2009
New Revision: 824543

URL: http://svn.apache.org/viewvc?rev=824543&view=rev
Log:
Use JDK Logger for Jetty

Modified:
    tuscany/java/sca/modules/host-jetty/src/main/java/org/apache/tuscany/sca/http/jetty/JettyLogger.java
    tuscany/java/sca/modules/host-jetty/src/main/java/org/apache/tuscany/sca/http/jetty/JettyServer.java

Modified: tuscany/java/sca/modules/host-jetty/src/main/java/org/apache/tuscany/sca/http/jetty/JettyLogger.java
URL: http://svn.apache.org/viewvc/tuscany/java/sca/modules/host-jetty/src/main/java/org/apache/tuscany/sca/http/jetty/JettyLogger.java?rev=824543&r1=824542&r2=824543&view=diff
==============================================================================
--- tuscany/java/sca/modules/host-jetty/src/main/java/org/apache/tuscany/sca/http/jetty/JettyLogger.java (original)
+++ tuscany/java/sca/modules/host-jetty/src/main/java/org/apache/tuscany/sca/http/jetty/JettyLogger.java Mon Oct 12 23:47:12 2009
@@ -18,6 +18,8 @@
  */
 package org.apache.tuscany.sca.http.jetty;
 
+import java.util.logging.Level;
+
 import org.mortbay.log.Logger;
 
 /**
@@ -26,62 +28,80 @@
  * @version $Rev$ $Date$
  */
 public class JettyLogger implements Logger {
+    private java.util.logging.Logger jdkLogger;
 
-    private boolean debugEnabled;
+    public JettyLogger() {
+        this(JettyLogger.class.getName());
+    }
 
-    public boolean isDebugEnabled() {
-        return debugEnabled;
+    public JettyLogger(String name) {
+        jdkLogger = java.util.logging.Logger.getLogger(name);
     }
 
-    public void setDebugEnabled(boolean debugEnabled) {
-        this.debugEnabled = debugEnabled;
+    /**
+     * @param jdkLogger
+     */
+    public JettyLogger(java.util.logging.Logger jdkLogger) {
+        super();
+        this.jdkLogger = jdkLogger;
     }
 
-    public void info(String msg, Object arg0, Object arg1) {
-        if (debugEnabled) {
-            System.err.println(":INFO:  " + format(msg, arg0, arg1));
-        }
+    public String getName() {
+        return jdkLogger.getName();
+    }
+
+    public void debug(String msg) {
+        jdkLogger.log(Level.FINE, msg);
     }
 
     public void debug(String msg, Throwable th) {
-        if (debugEnabled) {
-            System.err.println(":DEBUG:  " + msg);
-            th.printStackTrace();
-        }
+        jdkLogger.log(Level.FINE, msg, th);
     }
 
     public void debug(String msg, Object arg0, Object arg1) {
-        if (debugEnabled) {
-            System.err.println(":DEBUG: " + format(msg, arg0, arg1));
-        }
+        jdkLogger.log(Level.FINE, format(msg, arg0, arg1));
     }
 
-    public void warn(String msg, Object arg0, Object arg1) {
-        if (debugEnabled) {
-            System.err.println(":WARN: " + format(msg, arg0, arg1));
-        }
+    public Logger getLogger(String name) {
+        return new JettyLogger(name);
     }
 
-    public void warn(String msg, Throwable th) {
-        if (debugEnabled) {
-            System.err.println(":WARN: " + msg);
-            th.printStackTrace();
-        }
+    public void info(String msg) {
+        jdkLogger.log(Level.INFO, msg);
     }
 
-    public Logger getLogger(String name) {
-        return this;
+    public void info(String msg, Object arg0, Object arg1) {
+        jdkLogger.log(Level.INFO, format(msg, arg0, arg1));
+    }
+
+    public boolean isDebugEnabled() {
+        return jdkLogger.isLoggable(Level.FINE);
+    }
+
+    public void setDebugEnabled(boolean enabled) {
+        jdkLogger.setLevel(Level.FINE);
+    }
+
+    public void warn(String msg) {
+        jdkLogger.log(Level.WARNING, msg);
+    }
+
+    public void warn(String msg, Object arg0, Object arg1) {
+        jdkLogger.log(Level.WARNING, format(msg, arg0, arg1));
+    }
+
+    public void warn(String msg, Throwable th) {
+        jdkLogger.log(Level.WARNING, msg, th);
     }
 
     private String format(String msg, Object arg0, Object arg1) {
         int i0 = msg.indexOf("{}");
         int i1 = i0 < 0 ? -1 : msg.indexOf("{}", i0 + 2);
-        if (arg1 != null && i1 >= 0) {
+
+        if (arg1 != null && i1 >= 0)
             msg = msg.substring(0, i1) + arg1 + msg.substring(i1 + 2);
-        }
-        if (arg0 != null && i0 >= 0) {
+        if (arg0 != null && i0 >= 0)
             msg = msg.substring(0, i0) + arg0 + msg.substring(i0 + 2);
-        }
         return msg;
     }
 }

Modified: tuscany/java/sca/modules/host-jetty/src/main/java/org/apache/tuscany/sca/http/jetty/JettyServer.java
URL: http://svn.apache.org/viewvc/tuscany/java/sca/modules/host-jetty/src/main/java/org/apache/tuscany/sca/http/jetty/JettyServer.java?rev=824543&r1=824542&r2=824543&view=diff
==============================================================================
--- tuscany/java/sca/modules/host-jetty/src/main/java/org/apache/tuscany/sca/http/jetty/JettyServer.java (original)
+++ tuscany/java/sca/modules/host-jetty/src/main/java/org/apache/tuscany/sca/http/jetty/JettyServer.java Mon Oct 12 23:47:12 2009
@@ -509,12 +509,11 @@
 
     public void start() {
         try {
-            jettyLogger = Log.getLog();
+            jettyLogger = new JettyLogger(logger);
+            Log.setLog(jettyLogger);
         } catch (Throwable e) {
             // Ignore
-        } finally {
-            Log.setLog(new JettyLogger());
-        }
+        } 
     }
 
 }