Author: sebb
Date: Mon Oct 24 16:49:54 2011
New Revision: 1188223

URL: http://svn.apache.org/viewvc?rev=1188223&view=rev
Log:
Generic for loops

Modified:
    
jakarta/jmeter/trunk/src/core/org/apache/jmeter/engine/util/CompoundVariable.java
    
jakarta/jmeter/trunk/src/core/org/apache/jmeter/engine/util/ReplaceFunctionsWithStrings.java
    
jakarta/jmeter/trunk/src/core/org/apache/jmeter/engine/util/UndoVariableReplacement.java
    
jakarta/jmeter/trunk/src/core/org/apache/jmeter/engine/util/ValueReplacer.java

Modified: 
jakarta/jmeter/trunk/src/core/org/apache/jmeter/engine/util/CompoundVariable.java
URL: 
http://svn.apache.org/viewvc/jakarta/jmeter/trunk/src/core/org/apache/jmeter/engine/util/CompoundVariable.java?rev=1188223&r1=1188222&r2=1188223&view=diff
==============================================================================
--- 
jakarta/jmeter/trunk/src/core/org/apache/jmeter/engine/util/CompoundVariable.java
 (original)
+++ 
jakarta/jmeter/trunk/src/core/org/apache/jmeter/engine/util/CompoundVariable.java
 Mon Oct 24 16:49:54 2011
@@ -134,9 +134,7 @@ public class CompoundVariable implements
         }
         boolean testDynamic = false;
         StringBuilder results = new StringBuilder();
-        Iterator<Object> iter = compiledComponents.iterator();
-        while (iter.hasNext()) {
-            Object item = iter.next();
+        for (Object item : compiledComponents) {
             if (item instanceof Function) {
                 testDynamic = true;
                 try {

Modified: 
jakarta/jmeter/trunk/src/core/org/apache/jmeter/engine/util/ReplaceFunctionsWithStrings.java
URL: 
http://svn.apache.org/viewvc/jakarta/jmeter/trunk/src/core/org/apache/jmeter/engine/util/ReplaceFunctionsWithStrings.java?rev=1188223&r1=1188222&r2=1188223&view=diff
==============================================================================
--- 
jakarta/jmeter/trunk/src/core/org/apache/jmeter/engine/util/ReplaceFunctionsWithStrings.java
 (original)
+++ 
jakarta/jmeter/trunk/src/core/org/apache/jmeter/engine/util/ReplaceFunctionsWithStrings.java
 Mon Oct 24 16:49:54 2011
@@ -21,8 +21,8 @@
  */
 package org.apache.jmeter.engine.util;
 
-import java.util.Iterator;
 import java.util.Map;
+import java.util.Map.Entry;
 
 import org.apache.jmeter.functions.InvalidVariableException;
 import org.apache.jmeter.testelement.property.JMeterProperty;
@@ -69,11 +69,10 @@ public class ReplaceFunctionsWithStrings
         PatternMatcher pm = JMeterUtils.getMatcher();
         Pattern pattern = null;
         PatternCompiler compiler = new Perl5Compiler();
-        Iterator<String> iter = getVariables().keySet().iterator();
         String input = prop.getStringValue();
-        while (iter.hasNext()) {
-            String key = iter.next();
-            String value = getVariables().get(key);
+        for(Entry<String, String> entry : getVariables().entrySet()){
+            String key = entry.getKey();
+            String value = entry.getValue();
             if (regexMatch) {
                 try {
                     pattern = compiler.compile(value);

Modified: 
jakarta/jmeter/trunk/src/core/org/apache/jmeter/engine/util/UndoVariableReplacement.java
URL: 
http://svn.apache.org/viewvc/jakarta/jmeter/trunk/src/core/org/apache/jmeter/engine/util/UndoVariableReplacement.java?rev=1188223&r1=1188222&r2=1188223&view=diff
==============================================================================
--- 
jakarta/jmeter/trunk/src/core/org/apache/jmeter/engine/util/UndoVariableReplacement.java
 (original)
+++ 
jakarta/jmeter/trunk/src/core/org/apache/jmeter/engine/util/UndoVariableReplacement.java
 Mon Oct 24 16:49:54 2011
@@ -21,7 +21,6 @@
  */
 package org.apache.jmeter.engine.util;
 
-import java.util.Iterator;
 import java.util.Map;
 
 import org.apache.jmeter.functions.InvalidVariableException;
@@ -37,11 +36,10 @@ public class UndoVariableReplacement ext
     }
 
     public JMeterProperty transformValue(JMeterProperty prop) throws 
InvalidVariableException {
-        Iterator<String> iter = getVariables().keySet().iterator();
         String input = prop.getStringValue();
-        while (iter.hasNext()) {
-            String key = iter.next();
-            String value = getVariables().get(key);
+        for (Map.Entry<String, String> entry : getVariables().entrySet()) {
+            String key = entry.getKey();
+            String value = entry.getValue();
             input = StringUtilities.substitute(input, "${" + key + "}", value);
         }
         StringProperty newProp = new StringProperty(prop.getName(), input);

Modified: 
jakarta/jmeter/trunk/src/core/org/apache/jmeter/engine/util/ValueReplacer.java
URL: 
http://svn.apache.org/viewvc/jakarta/jmeter/trunk/src/core/org/apache/jmeter/engine/util/ValueReplacer.java?rev=1188223&r1=1188222&r2=1188223&view=diff
==============================================================================
--- 
jakarta/jmeter/trunk/src/core/org/apache/jmeter/engine/util/ValueReplacer.java 
(original)
+++ 
jakarta/jmeter/trunk/src/core/org/apache/jmeter/engine/util/ValueReplacer.java 
Mon Oct 24 16:49:54 2011
@@ -20,7 +20,6 @@ package org.apache.jmeter.engine.util;
 
 import java.util.Collection;
 import java.util.HashMap;
-import java.util.Iterator;
 import java.util.LinkedList;
 import java.util.List;
 import java.util.Map;
@@ -68,9 +67,8 @@ public class ValueReplacer {
 
     private void setProperties(TestElement el, Collection<JMeterProperty> 
newProps) {
         el.clear();
-        Iterator<JMeterProperty> iter = newProps.iterator();
-        while (iter.hasNext()) {
-            el.setProperty(iter.next());
+        for (JMeterProperty jmp : newProps) {
+            el.setProperty(jmp);
         }
     }
 
@@ -126,9 +124,8 @@ public class ValueReplacer {
                 MultiProperty multiVal = (MultiProperty) val;
                 Collection<JMeterProperty> newValues = 
replaceValues(multiVal.iterator(), transform);
                 multiVal.clear();
-                Iterator<JMeterProperty> propIter = newValues.iterator();
-                while (propIter.hasNext()) {
-                    multiVal.addProperty(propIter.next());
+                for (JMeterProperty jmp : newValues) {
+                    multiVal.addProperty(jmp);
                 }
                 if (log.isDebugEnabled()) {
                     log.debug("Replacement result: " + multiVal);



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to