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 {