You are viewing a plain text version of this content. The canonical link for it is here.

Posted to scm@geronimo.apache.org by ga...@apache.org on 2010/11/17 19:09:40 UTC

svn commit: r1036151 - /geronimo/server/trunk/framework/modules/geronimo-hook/src/main/java/org/apache/geronimo/hook/equinox/ClassLoaderHook.java

Author: gawor
Date: Wed Nov 17 18:09:40 2010
New Revision: 1036151

URL: http://svn.apache.org/viewvc?rev=1036151&view=rev
Log:
ensure we are adding a non-empty package list

Modified:
    geronimo/server/trunk/framework/modules/geronimo-hook/src/main/java/org/apache/geronimo/hook/equinox/ClassLoaderHook.java

Modified: geronimo/server/trunk/framework/modules/geronimo-hook/src/main/java/org/apache/geronimo/hook/equinox/ClassLoaderHook.java
URL: http://svn.apache.org/viewvc/geronimo/server/trunk/framework/modules/geronimo-hook/src/main/java/org/apache/geronimo/hook/equinox/ClassLoaderHook.java?rev=1036151&r1=1036150&r2=1036151&view=diff
==============================================================================
--- geronimo/server/trunk/framework/modules/geronimo-hook/src/main/java/org/apache/geronimo/hook/equinox/ClassLoaderHook.java (original)
+++ geronimo/server/trunk/framework/modules/geronimo-hook/src/main/java/org/apache/geronimo/hook/equinox/ClassLoaderHook.java Wed Nov 17 18:09:40 2010
@@ -45,7 +45,9 @@ public class ClassLoaderHook implements 
         Collections.synchronizedMap(new HashMap<Long, String>());
     
     public void addDynamicImportPackage(long bundleId, String packages) {
-        dynamicPackages.put(bundleId, packages);
+        if (packages != null && packages.trim().length() > 0) {
+            dynamicPackages.put(bundleId, packages);
+        }
     }
     
     public void removeDynamicImportPackage(long bundleId) {