commit a059dabc33eaff0b1f710d16054743624647f295
author Carsten Ziegeler <cziegeler@apache.org> Mon Sep 16 11:54:54 2013 +0000
committer Carsten Ziegeler <cziegeler@apache.org> Mon Sep 16 11:54:54 2013 +0000
tree 749335bc58fcd1ee82d6bd708263aa4e7218389a
parent 6ffa856e3a1666ad790d2072699bc171ac7b6de7

SLING-3071 : DynamicClassLoaderManager should return DynamicClassLoaders. Apply patch from Francesco Mari.

git-svn-id: https://svn.apache.org/repos/asf/sling/trunk@1523603 13f79535-47bb-0310-9956-ffa450edef68

src/main/java/org/apache/sling/commons/classloader/impl/ClassLoaderFacade.java

diff --git a/src/main/java/org/apache/sling/commons/classloader/impl/ClassLoaderFacade.java b/src/main/java/org/apache/sling/commons/classloader/impl/ClassLoaderFacade.java
index 84f54bd..f60a34b 100644
--- a/src/main/java/org/apache/sling/commons/classloader/impl/ClassLoaderFacade.java
+++ b/src/main/java/org/apache/sling/commons/classloader/impl/ClassLoaderFacade.java
@@ -25,6 +25,7 @@
 import java.util.Enumeration;
 import java.util.List;
 
+import org.apache.sling.commons.classloader.DynamicClassLoader;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
@@ -36,14 +37,14 @@
  * This class loader delegates to other class loaders
  * but caches its result for performance.
  */
-public class ClassLoaderFacade extends ClassLoader {
+public class ClassLoaderFacade extends ClassLoader implements DynamicClassLoader {
 
     /** The logger. */
     private final Logger logger = LoggerFactory.getLogger(this.getClass().getName());
 
     /** Dynamic class loader manager which manages the dynamic class loader providers for this facade. */
     private final DynamicClassLoaderManagerImpl manager;
-    
+
     /** Used to log stack traces in slf4j for non-critical errors */
     @SuppressWarnings("serial")
     static class StackTraceProbe extends Exception {
@@ -60,11 +61,11 @@
     }
 
     /** Return false if our manager is not active, and log the stack trace
-     *  when that happens, to ease troubleshooting. 
+     *  when that happens, to ease troubleshooting.
      */
     private boolean checkManagerActive() {
         if(!this.manager.isActive()) {
-            final String msg = "Dynamic class loader has already been deactivated."; 
+            final String msg = "Dynamic class loader has already been deactivated.";
             final StackTraceProbe p = new StackTraceProbe(msg);
             logger.error(msg, p);
             return false;
@@ -75,6 +76,7 @@
     /**
      * @see java.lang.ClassLoader#getResource(java.lang.String)
      */
+    @Override
     public URL getResource(String name) {
         if (!checkManagerActive()) {
             return null;
@@ -100,6 +102,7 @@
     /**
      * @see java.lang.ClassLoader#getResources(java.lang.String)
      */
+    @Override
     public Enumeration<URL> getResources(String name) throws IOException {
         if(!checkManagerActive()) {
             return Collections.enumeration(EMPTY_LIST);
@@ -124,6 +127,7 @@
     /**
      * @see java.lang.ClassLoader#loadClass(java.lang.String, boolean)
      */
+    @Override
     protected synchronized Class<?> loadClass(String name, boolean resolve)
     throws ClassNotFoundException {
         if(!checkManagerActive()) {
@@ -144,4 +148,11 @@
         }
         throw new ClassNotFoundException(name);
     }
+
+    /**
+     * @see org.apache.sling.commons.classloader.DynamicClassLoader#isLive()
+     */
+    public boolean isLive() {
+        return this.manager.isActive();
+    }
 }