Author: sebb
Date: Mon Oct 24 16:03:51 2011
New Revision: 1188186

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

Modified:
    jakarta/jmeter/trunk/test/src/org/apache/jmeter/resources/PackageTest.java
    
jakarta/jmeter/trunk/test/src/org/apache/jmeter/testbeans/gui/PackageTest.java
    jakarta/jmeter/trunk/test/src/org/apache/jorphan/test/AllTests.java

Modified: 
jakarta/jmeter/trunk/test/src/org/apache/jmeter/resources/PackageTest.java
URL: 
http://svn.apache.org/viewvc/jakarta/jmeter/trunk/test/src/org/apache/jmeter/resources/PackageTest.java?rev=1188186&r1=1188185&r2=1188186&view=diff
==============================================================================
--- jakarta/jmeter/trunk/test/src/org/apache/jmeter/resources/PackageTest.java 
(original)
+++ jakarta/jmeter/trunk/test/src/org/apache/jmeter/resources/PackageTest.java 
Mon Oct 24 16:03:51 2011
@@ -183,10 +183,8 @@ public class PackageTest extends TestCas
             }
             final ArrayList<String> list = Collections.list(prb.getKeys());
             Collections.sort(list);
-            Iterator<String> enumr = list.iterator();
             final boolean mainResourceFile = resname.startsWith("messages");
-            while (enumr.hasNext()) {
-                String key = enumr.next();
+            for (String key : list) {
                 try {
                     String val = defaultPRB.getString(key); // Also Check key 
is in default
                     if (mainResourceFile && val.equals(prb.getString(key))){

Modified: 
jakarta/jmeter/trunk/test/src/org/apache/jmeter/testbeans/gui/PackageTest.java
URL: 
http://svn.apache.org/viewvc/jakarta/jmeter/trunk/test/src/org/apache/jmeter/testbeans/gui/PackageTest.java?rev=1188186&r1=1188185&r2=1188186&view=diff
==============================================================================
--- 
jakarta/jmeter/trunk/test/src/org/apache/jmeter/testbeans/gui/PackageTest.java 
(original)
+++ 
jakarta/jmeter/trunk/test/src/org/apache/jmeter/testbeans/gui/PackageTest.java 
Mon Oct 24 16:03:51 2011
@@ -22,6 +22,7 @@ import java.beans.Introspector;
 import java.beans.PropertyDescriptor;
 import java.util.Enumeration;
 import java.util.Iterator;
+import java.util.List;
 import java.util.Locale;
 import java.util.ResourceBundle;
 
@@ -167,13 +168,11 @@ public class PackageTest extends JMeterT
     public static Test suite() throws Exception {
         TestSuite suite = new TestSuite("Bean Resource Test Suite");
 
-        Iterator<String> iter = 
ClassFinder.findClassesThatExtend(JMeterUtils.getSearchPaths(), new Class[] { 
TestBean.class })
-                .iterator();
+        List<String> testBaeanclassNames = 
ClassFinder.findClassesThatExtend(JMeterUtils.getSearchPaths(), new Class[] { 
TestBean.class });
 
         boolean errorDetected = false;
         JMeterUtils.setLocale(defaultLocale);
-        while (iter.hasNext()) {
-            String className = iter.next();
+        for (String className : testBaeanclassNames) {
             Class<?> testBeanClass = Class.forName(className);
             ResourceBundle defaultBundle = null;
             try {

Modified: jakarta/jmeter/trunk/test/src/org/apache/jorphan/test/AllTests.java
URL: 
http://svn.apache.org/viewvc/jakarta/jmeter/trunk/test/src/org/apache/jorphan/test/AllTests.java?rev=1188186&r1=1188185&r2=1188186&view=diff
==============================================================================
--- jakarta/jmeter/trunk/test/src/org/apache/jorphan/test/AllTests.java 
(original)
+++ jakarta/jmeter/trunk/test/src/org/apache/jorphan/test/AllTests.java Mon Oct 
24 16:03:51 2011
@@ -330,9 +330,7 @@ public final class AllTests {
             int sz=classList.size();
             log.info("ClassFinder(TestCase) found: "+sz+ " TestCase classes");
             System.out.println("ClassFinder found: "+sz+ " TestCase classes");
-            Iterator<String> classes = classList.iterator();
-            while (classes.hasNext()) {
-                String name = classes.next();
+            for (String name : classList) {
                 try {
                     /*
                      * TestSuite only finds testXXX() methods, and does not 
look



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

Reply via email to