]> source.dussan.org Git - pf4j.git/commitdiff
Remove class loaders checking
authorDecebal Suiu <decebal.suiu@gmail.com>
Fri, 1 Jul 2016 21:01:38 +0000 (00:01 +0300)
committerDecebal Suiu <decebal.suiu@gmail.com>
Fri, 1 Jul 2016 21:01:38 +0000 (00:01 +0300)
pf4j/src/main/java/ro/fortsoft/pf4j/AbstractExtensionFinder.java
pf4j/src/main/java/ro/fortsoft/pf4j/PluginClassLoader.java

index 900ebb90f7c2f5da7501b56107b88b00463f567f..6d3180b5a2a3251edb4143381f4c30dbca242bad 100644 (file)
@@ -70,20 +70,6 @@ public abstract class AbstractExtensionFinder implements ExtensionFinder, Plugin
 
             ClassLoader classLoader = (pluginId != null) ? pluginManager.getPluginClassLoader(pluginId) : getClass().getClassLoader();
 
-            // performs some check
-            if (log.isTraceEnabled()) { // an alternative may be the use of a system property
-                ClassLoader typeClassLoader = type.getClassLoader();
-                if (!classLoader.equals(typeClassLoader))
-                {
-                    // in this scenario the method 'isAssignableFrom' returns only FALSE
-                    // see http://www.coderanch.com/t/557846/java/java/FWIW-FYI-isAssignableFrom-isInstance-differing
-                    log.error("Different class loaders: '{}' (E) and '{}' (EP)", classLoader, typeClassLoader);
-
-                    // below line is commented because I wish to see the next log.trace line
-//                    continue;
-                }
-            }
-
             for (String className : entry.getValue()) {
                 try {
                     log.debug("Loading class '{}' using class loader '{}'", className, classLoader);
index 34259af2739449c9de098260008788f27c1646f0..093623bea6202d65edd4a98e234825ab7b269b23 100644 (file)
@@ -90,7 +90,7 @@ public class PluginClassLoader extends URLClassLoader {
             }
 
             // look in dependencies
-            log.trace("Look in dependencies for class '{}'", className);
+            log.trace("Search in dependencies for class '{}'", className);
             List<PluginDependency> dependencies = pluginDescriptor.getDependencies();
             for (PluginDependency dependency : dependencies) {
                 PluginClassLoader classLoader = pluginManager.getPluginClassLoader(dependency.getPluginId());