You are viewing a plain text version of this content. The canonical link for it is here.
Posted to scm@geronimo.apache.org by jd...@apache.org on 2008/10/20 10:20:36 UTC

svn commit: r706146 - in /geronimo/gshell/trunk/gshell-bootstrap/src/main/java/org/apache/geronimo/gshell/bootstrap: Configuration.java Launcher.java

Author: jdillon
Date: Mon Oct 20 01:20:36 2008
New Revision: 706146

URL: http://svn.apache.org/viewvc?rev=706146&view=rev
Log:
A few trivial logging optimizations

Modified:
    geronimo/gshell/trunk/gshell-bootstrap/src/main/java/org/apache/geronimo/gshell/bootstrap/Configuration.java
    geronimo/gshell/trunk/gshell-bootstrap/src/main/java/org/apache/geronimo/gshell/bootstrap/Launcher.java

Modified: geronimo/gshell/trunk/gshell-bootstrap/src/main/java/org/apache/geronimo/gshell/bootstrap/Configuration.java
URL: http://svn.apache.org/viewvc/geronimo/gshell/trunk/gshell-bootstrap/src/main/java/org/apache/geronimo/gshell/bootstrap/Configuration.java?rev=706146&r1=706145&r2=706146&view=diff
==============================================================================
--- geronimo/gshell/trunk/gshell-bootstrap/src/main/java/org/apache/geronimo/gshell/bootstrap/Configuration.java (original)
+++ geronimo/gshell/trunk/gshell-bootstrap/src/main/java/org/apache/geronimo/gshell/bootstrap/Configuration.java Mon Oct 20 01:20:36 2008
@@ -125,7 +125,10 @@
 
     private void setProperty(final String name, final String value) {
         System.setProperty(name, value);
-        Log.debug("Property: " + name + "=" + value);
+
+        if (Log.DEBUG) {
+            Log.debug("Property: " + name + "=" + value);
+        }
     }
 
     public void configure() throws Exception {

Modified: geronimo/gshell/trunk/gshell-bootstrap/src/main/java/org/apache/geronimo/gshell/bootstrap/Launcher.java
URL: http://svn.apache.org/viewvc/geronimo/gshell/trunk/gshell-bootstrap/src/main/java/org/apache/geronimo/gshell/bootstrap/Launcher.java?rev=706146&r1=706145&r2=706146&view=diff
==============================================================================
--- geronimo/gshell/trunk/gshell-bootstrap/src/main/java/org/apache/geronimo/gshell/bootstrap/Launcher.java (original)
+++ geronimo/gshell/trunk/gshell-bootstrap/src/main/java/org/apache/geronimo/gshell/bootstrap/Launcher.java Mon Oct 20 01:20:36 2008
@@ -51,7 +51,9 @@
 
             Thread.currentThread().setContextClassLoader(cl);
 
-            Log.debug("Launching: " + method);
+            if (Log.DEBUG) {
+                Log.debug("Launching: " + method);
+            }
 
             method.invoke(null, new Object[] { args });