You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@logging.apache.org by ma...@apache.org on 2014/10/08 04:02:45 UTC

[2/3] git commit: Use LoaderUtil.loadClass instead of getTCCL

Use LoaderUtil.loadClass instead of getTCCL


Project: http://git-wip-us.apache.org/repos/asf/logging-log4j2/repo
Commit: http://git-wip-us.apache.org/repos/asf/logging-log4j2/commit/c17edce9
Tree: http://git-wip-us.apache.org/repos/asf/logging-log4j2/tree/c17edce9
Diff: http://git-wip-us.apache.org/repos/asf/logging-log4j2/diff/c17edce9

Branch: refs/heads/master
Commit: c17edce96f6cc5585d86b9a5f2a5e619aa55f2b0
Parents: 3df43dd
Author: Matt Sicker <ma...@apache.org>
Authored: Tue Oct 7 21:01:12 2014 -0500
Committer: Matt Sicker <ma...@apache.org>
Committed: Tue Oct 7 21:01:12 2014 -0500

----------------------------------------------------------------------
 .../src/main/java/org/apache/logging/log4j/LogManager.java      | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/logging-log4j2/blob/c17edce9/log4j-api/src/main/java/org/apache/logging/log4j/LogManager.java
----------------------------------------------------------------------
diff --git a/log4j-api/src/main/java/org/apache/logging/log4j/LogManager.java b/log4j-api/src/main/java/org/apache/logging/log4j/LogManager.java
index 0f866b8..5eb03a5 100644
--- a/log4j-api/src/main/java/org/apache/logging/log4j/LogManager.java
+++ b/log4j-api/src/main/java/org/apache/logging/log4j/LogManager.java
@@ -71,12 +71,11 @@ public class LogManager {
         // Shortcut binding to force a specific logging implementation.
         final PropertiesUtil managerProps = PropertiesUtil.getProperties();
         final String factoryClassName = managerProps.getStringProperty(FACTORY_PROPERTY_NAME);
-        final ClassLoader cl = LoaderUtil.getThreadContextClassLoader();
         if (factoryClassName != null) {
             try {
-                final Class<?> clazz = cl.loadClass(factoryClassName);
+                final Class<?> clazz = LoaderUtil.loadClass(factoryClassName);
                 if (LoggerContextFactory.class.isAssignableFrom(clazz)) {
-                    factory = (LoggerContextFactory) clazz.newInstance();
+                    factory = clazz.asSubclass(LoggerContextFactory.class).newInstance();
                 }
             } catch (final ClassNotFoundException cnfe) {
                 LOGGER.error("Unable to locate configured LoggerContextFactory {}", factoryClassName);