Author: scheu
Date: Mon Nov 15 16:20:16 2010
New Revision: 1035340

URL: http://svn.apache.org/viewvc?rev=1035340&view=rev
Log:
AXIS2-4844
Contributor:Rich Scheuerle
Avoid iterator usage in this frequently called method.

Modified:
    
axis/axis2/java/core/trunk/modules/kernel/src/org/apache/axis2/util/ThreadContextMigratorUtil.java

Modified: 
axis/axis2/java/core/trunk/modules/kernel/src/org/apache/axis2/util/ThreadContextMigratorUtil.java
URL: 
http://svn.apache.org/viewvc/axis/axis2/java/core/trunk/modules/kernel/src/org/apache/axis2/util/ThreadContextMigratorUtil.java?rev=1035340&r1=1035339&r2=1035340&view=diff
==============================================================================
--- 
axis/axis2/java/core/trunk/modules/kernel/src/org/apache/axis2/util/ThreadContextMigratorUtil.java
 (original)
+++ 
axis/axis2/java/core/trunk/modules/kernel/src/org/apache/axis2/util/ThreadContextMigratorUtil.java
 Mon Nov 15 16:20:16 2010
@@ -27,7 +27,6 @@ import org.apache.axis2.engine.AxisConfi
 
 import java.util.LinkedList;
 import java.util.List;
-import java.util.ListIterator;
 
 /**
  * This is a utility class to make it easier/cleaner for user programming
@@ -99,10 +98,10 @@ public class ThreadContextMigratorUtil {
 
         if (param != null) {
             List migratorList = (List) param.getValue();
-            ListIterator threadContextMigrators = migratorList.listIterator();
-            while (threadContextMigrators.hasNext()) {
-                ((ThreadContextMigrator) threadContextMigrators.next())
-                        .migrateContextToThread(msgContext);
+            int size = migratorList.size();
+            for (int i=0; i<size;i++) {
+                ((ThreadContextMigrator) migratorList.get(i))
+                .migrateContextToThread(msgContext);
             }
         }
     }
@@ -128,9 +127,9 @@ public class ThreadContextMigratorUtil {
 
         if (param != null) {
             List migratorList = (List) param.getValue();
-            ListIterator threadContextMigrators = migratorList.listIterator();
-            while (threadContextMigrators.hasNext()) {
-                ((ThreadContextMigrator) 
threadContextMigrators.next()).cleanupThread(msgContext);
+            int size = migratorList.size();
+            for (int i=0; i<size;i++) {
+                ((ThreadContextMigrator) 
migratorList.get(i)).cleanupThread(msgContext);
             }
         }
     }
@@ -158,9 +157,9 @@ public class ThreadContextMigratorUtil {
 
         if (param != null) {
             List migratorList = (List) param.getValue();
-            ListIterator threadContextMigrators = migratorList.listIterator();
-            while (threadContextMigrators.hasNext()) {
-                ((ThreadContextMigrator) threadContextMigrators.next())
+            int size = migratorList.size();
+            for (int i=0; i<size;i++) {
+                ((ThreadContextMigrator) migratorList.get(i))
                         .migrateThreadToContext(msgContext);
             }
         }
@@ -187,9 +186,9 @@ public class ThreadContextMigratorUtil {
 
         if (param != null) {
             List migratorList = (List) param.getValue();
-            ListIterator threadContextMigrators = migratorList.listIterator();
-            while (threadContextMigrators.hasNext()) {
-                ((ThreadContextMigrator) 
threadContextMigrators.next()).cleanupContext(msgContext);
+            int size = migratorList.size();
+            for (int i=0; i<size;i++) {
+                ((ThreadContextMigrator) 
migratorList.get(i)).cleanupContext(msgContext);
             }
         }
   }


Reply via email to