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 2005/05/24 16:32:16 UTC

svn commit: r178169 - /jakarta/commons/proper/logging/branches/simon-1.1/src/java/org/apache/commons/logging/LogFactory.java.template

Author: skitching
Date: Tue May 24 07:32:15 2005
New Revision: 178169

URL: http://svn.apache.org/viewcvs?rev=178169&view=rev
Log:
Remove special-casing for ContextFactory. Instead of setting the
default factory based on the class the LogFactory is bound to,
ContextFactory uses a system property. This is now allows
ContextFactory to be just another Factory implementation for
LogFactory to bind to statically.

Modified:
    jakarta/commons/proper/logging/branches/simon-1.1/src/java/org/apache/commons/logging/LogFactory.java.template

Modified: jakarta/commons/proper/logging/branches/simon-1.1/src/java/org/apache/commons/logging/LogFactory.java.template
URL: http://svn.apache.org/viewcvs/jakarta/commons/proper/logging/branches/simon-1.1/src/java/org/apache/commons/logging/LogFactory.java.template?rev=178169&r1=178168&r2=178169&view=diff
==============================================================================
--- jakarta/commons/proper/logging/branches/simon-1.1/src/java/org/apache/commons/logging/LogFactory.java.template (original)
+++ jakarta/commons/proper/logging/branches/simon-1.1/src/java/org/apache/commons/logging/LogFactory.java.template Tue May 24 07:32:15 2005
@@ -18,7 +18,6 @@
 
 import org.apache.commons.logging.Log;
 import org.apache.commons.logging.Factory;
-import org.apache.commons.logging.ContextFactory;
 
 
 /**
@@ -40,17 +39,15 @@
          * The actual class used here is modified during compilation.
          */
         factory = new org.apache.commons.logging.impl.@IMPL@Factory();
-        
-        String context = System.getProperty("org.apache.commons.logging.context");
-        if (Boolean.parseBoolean(context)) {
-            // presumably this class is being used within a container, and
-            // has been deployed via a shared classloader
-            factory = new ContextFactory(factory);
-        }
     }
 
     // --------------------------------------------------------- Static Methods
 
+    /**
+     * Return the underlying factory object that this LogFactory class
+     * is bound to. This method is intended for the internal use of
+     * commons-logging and should not be called by user code.
+     */
     public static Factory getFactory() {
         return factory;
     }



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