You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@commons.apache.org by sk...@apache.org on 2006/03/09 04:03:52 UTC

svn commit: r384411 - in /jakarta/commons/proper/logging/contrib/simon/jcl2: core-dynamic/java/org/apache/commons/logging/ core-static/java/org/apache/commons/logging/ noop/java/org/apache/commons/logging/noop/ simple/java/org/apache/commons/logging/si...

Author: skitching
Date: Wed Mar  8 19:03:49 2006
New Revision: 384411

URL: http://svn.apache.org/viewcvs?rev=384411&view=rev
Log:
Method renames due to change in core; no functional changes.

Modified:
    jakarta/commons/proper/logging/contrib/simon/jcl2/core-dynamic/java/org/apache/commons/logging/LogFactoryDynamic.java
    jakarta/commons/proper/logging/contrib/simon/jcl2/core-static/java/org/apache/commons/logging/LogFactoryStatic.java
    jakarta/commons/proper/logging/contrib/simon/jcl2/noop/java/org/apache/commons/logging/noop/NoOpLogHandler.java
    jakarta/commons/proper/logging/contrib/simon/jcl2/simple/java/org/apache/commons/logging/simple/SimpleLogHandler.java

Modified: jakarta/commons/proper/logging/contrib/simon/jcl2/core-dynamic/java/org/apache/commons/logging/LogFactoryDynamic.java
URL: http://svn.apache.org/viewcvs/jakarta/commons/proper/logging/contrib/simon/jcl2/core-dynamic/java/org/apache/commons/logging/LogFactoryDynamic.java?rev=384411&r1=384410&r2=384411&view=diff
==============================================================================
--- jakarta/commons/proper/logging/contrib/simon/jcl2/core-dynamic/java/org/apache/commons/logging/LogFactoryDynamic.java (original)
+++ jakarta/commons/proper/logging/contrib/simon/jcl2/core-dynamic/java/org/apache/commons/logging/LogFactoryDynamic.java Wed Mar  8 19:03:49 2006
@@ -51,8 +51,8 @@
      *
      * @param clazz Class from which a log name will be derived
      */
-    public Log getInstance(Class clazz) {
-        return getHandler().getLog(clazz);
+    public Log getLogFor(Class clazz) {
+        return getHandler().getLogFor(clazz);
     }
 
     /**
@@ -60,12 +60,12 @@
      *  returned (the meaning of this name is only known to the underlying
      *  logging implementation that is being wrapped)
      */
-    public Log getInstance(String name) {
-        return getHandler().getLog(name);
+    public Log getLogFor(String name) {
+        return getHandler().getLogFor(name);
     }
     
-    public void releaseInstance() {
-        getHandler().release();
+    public void releaseResources() {
+        getHandler().releaseResources();
     }
     
     private synchronized LogHandler getHandler() {

Modified: jakarta/commons/proper/logging/contrib/simon/jcl2/core-static/java/org/apache/commons/logging/LogFactoryStatic.java
URL: http://svn.apache.org/viewcvs/jakarta/commons/proper/logging/contrib/simon/jcl2/core-static/java/org/apache/commons/logging/LogFactoryStatic.java?rev=384411&r1=384410&r2=384411&view=diff
==============================================================================
--- jakarta/commons/proper/logging/contrib/simon/jcl2/core-static/java/org/apache/commons/logging/LogFactoryStatic.java (original)
+++ jakarta/commons/proper/logging/contrib/simon/jcl2/core-static/java/org/apache/commons/logging/LogFactoryStatic.java Wed Mar  8 19:03:49 2006
@@ -51,8 +51,8 @@
      *
      * @param clazz Class from which a log name will be derived
      */
-    public Log getInstance(Class clazz) {
-        return handler.getLog(clazz);
+    public Log getLogFor(Class clazz) {
+        return handler.getLogFor(clazz);
     }
 
     /**
@@ -60,11 +60,11 @@
      *  returned (the meaning of this name is only known to the underlying
      *  logging implementation that is being wrapped)
      */
-    public Log getInstance(String name) {
-        return handler.getLog(name);
+    public Log getLogFor(String name) {
+        return handler.getLogFor(name);
     }
     
-    public void releaseInstance() {
-        handler.release();
+    public void releaseResources() {
+        handler.releaseResources();
     }
 }

Modified: jakarta/commons/proper/logging/contrib/simon/jcl2/noop/java/org/apache/commons/logging/noop/NoOpLogHandler.java
URL: http://svn.apache.org/viewcvs/jakarta/commons/proper/logging/contrib/simon/jcl2/noop/java/org/apache/commons/logging/noop/NoOpLogHandler.java?rev=384411&r1=384410&r2=384411&view=diff
==============================================================================
--- jakarta/commons/proper/logging/contrib/simon/jcl2/noop/java/org/apache/commons/logging/noop/NoOpLogHandler.java (original)
+++ jakarta/commons/proper/logging/contrib/simon/jcl2/noop/java/org/apache/commons/logging/noop/NoOpLogHandler.java Wed Mar  8 19:03:49 2006
@@ -30,14 +30,14 @@
 public final class NoOpLogHandler implements LogHandler {
     private NoOpLog instance = new NoOpLog();
 
-    public Log getLog(Class clazz) {
+    public Log getLogFor(Class clazz) {
         return instance;
     }
 
-    public Log getLog(String category) {
+    public Log getLogFor(String category) {
         return instance;
     }
 
-    public void release() {
+    public void releaseResources() {
     }
 }

Modified: jakarta/commons/proper/logging/contrib/simon/jcl2/simple/java/org/apache/commons/logging/simple/SimpleLogHandler.java
URL: http://svn.apache.org/viewcvs/jakarta/commons/proper/logging/contrib/simon/jcl2/simple/java/org/apache/commons/logging/simple/SimpleLogHandler.java?rev=384411&r1=384410&r2=384411&view=diff
==============================================================================
--- jakarta/commons/proper/logging/contrib/simon/jcl2/simple/java/org/apache/commons/logging/simple/SimpleLogHandler.java (original)
+++ jakarta/commons/proper/logging/contrib/simon/jcl2/simple/java/org/apache/commons/logging/simple/SimpleLogHandler.java Wed Mar  8 19:03:49 2006
@@ -127,11 +127,11 @@
 
     // ----------- Public methods ----------------------------
 
-    public Log getLog(Class clazz) {
-        return getLog(clazz.getName());
+    public Log getLogFor(Class clazz) {
+        return getLogFor(clazz.getName());
     }
 
-    public synchronized Log getLog(String category) {
+    public synchronized Log getLogFor(String category) {
         if (logs == null) {
             init();
         }
@@ -147,7 +147,7 @@
     /**
      * Release all resources held by the simple logging facility.
      */
-    public synchronized void release() {
+    public synchronized void releaseResources() {
         logs = null;
     }
     
@@ -172,7 +172,9 @@
                 simpleLogProps.load(in);
                 in.close();
             } catch(java.io.IOException e) {
-                logDiagnostics("Unable to load file simplelog.properties");
+                // ok to output this info to stderr, because that's what
+                // this logging system does anyway
+                System.err.println("Unable to load file simplelog.properties");
             }
         }
 



---------------------------------------------------------------------
To unsubscribe, e-mail: commons-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: commons-dev-help@jakarta.apache.org