geronimo-scm mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From ke...@apache.org
Subject svn commit: r644146 - /geronimo/server/trunk/framework/modules/geronimo-transformer/src/main/java/org/apache/geronimo/transformer/TransformerCollection.java
Date Thu, 03 Apr 2008 03:18:22 GMT
Author: kevan
Date: Wed Apr  2 20:18:21 2008
New Revision: 644146

URL: http://svn.apache.org/viewvc?rev=644146&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/trunk/framework/modules/geronimo-transformer/src/main/java/org/apache/geronimo/transformer/TransformerCollection.java

Modified: geronimo/server/trunk/framework/modules/geronimo-transformer/src/main/java/org/apache/geronimo/transformer/TransformerCollection.java
URL: http://svn.apache.org/viewvc/geronimo/server/trunk/framework/modules/geronimo-transformer/src/main/java/org/apache/geronimo/transformer/TransformerCollection.java?rev=644146&r1=644145&r2=644146&view=diff
==============================================================================
--- geronimo/server/trunk/framework/modules/geronimo-transformer/src/main/java/org/apache/geronimo/transformer/TransformerCollection.java
(original)
+++ geronimo/server/trunk/framework/modules/geronimo-transformer/src/main/java/org/apache/geronimo/transformer/TransformerCollection.java
Wed Apr  2 20:18:21 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
{



Mime
View raw message