You are viewing a plain text version of this content. The canonical link for it is here.
Posted to scm@geronimo.apache.org by ke...@apache.org on 2008/04/03 05:20:27 UTC

svn commit: r644147 - /geronimo/server/branches/2.0/modules/geronimo-transformer/src/main/java/org/apache/geronimo/transformer/TransformerCollection.java

Author: kevan
Date: Wed Apr  2 20:20:27 2008
New Revision: 644147

URL: http://svn.apache.org/viewvc?rev=644147&view=rev
Log:
GERONIMO-3687 Avoid classloader deadlock. It seems that some JSE's aren't using .iterator() in an enhanced for loop. So, simple call to AbstractList.iterator() no longer works

Modified:
    geronimo/server/branches/2.0/modules/geronimo-transformer/src/main/java/org/apache/geronimo/transformer/TransformerCollection.java

Modified: geronimo/server/branches/2.0/modules/geronimo-transformer/src/main/java/org/apache/geronimo/transformer/TransformerCollection.java
URL: http://svn.apache.org/viewvc/geronimo/server/branches/2.0/modules/geronimo-transformer/src/main/java/org/apache/geronimo/transformer/TransformerCollection.java?rev=644147&r1=644146&r2=644147&view=diff
==============================================================================
--- geronimo/server/branches/2.0/modules/geronimo-transformer/src/main/java/org/apache/geronimo/transformer/TransformerCollection.java (original)
+++ geronimo/server/branches/2.0/modules/geronimo-transformer/src/main/java/org/apache/geronimo/transformer/TransformerCollection.java Wed Apr  2 20:20:27 2008
@@ -33,7 +33,8 @@
     // hack to force load of ArrayList$Itr class. This avoids a potential Classloader deadlock during startup
     // see GERONIMO-3687
     {
-        transformers.iterator();
+        for (ClassFileTransformer transformer : transformers) {
+        }
     }
 
     public byte[] transform(ClassLoader loader, String className, Class<?> classBeingRedefined, ProtectionDomain protectionDomain, byte[] classfileBuffer) throws IllegalClassFormatException {