Author: sebb
Date: Mon Oct 24 16:47:20 2011
New Revision: 1188218

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

Modified:
    jakarta/jmeter/trunk/src/core/org/apache/jmeter/util/JMeterUtils.java

Modified: jakarta/jmeter/trunk/src/core/org/apache/jmeter/util/JMeterUtils.java
URL: 
http://svn.apache.org/viewvc/jakarta/jmeter/trunk/src/core/org/apache/jmeter/util/JMeterUtils.java?rev=1188218&r1=1188217&r2=1188218&view=diff
==============================================================================
--- jakarta/jmeter/trunk/src/core/org/apache/jmeter/util/JMeterUtils.java 
(original)
+++ jakarta/jmeter/trunk/src/core/org/apache/jmeter/util/JMeterUtils.java Mon 
Oct 24 16:47:20 2011
@@ -32,7 +32,6 @@ import java.net.URL;
 import java.net.UnknownHostException;
 import java.util.Enumeration;
 import java.util.Hashtable;
-import java.util.Iterator;
 import java.util.LinkedHashMap;
 import java.util.List;
 import java.util.Locale;
@@ -392,10 +391,9 @@ public class JMeterUtils implements Unit
     private static void notifyLocaleChangeListeners() {
         LocaleChangeEvent event = new LocaleChangeEvent(JMeterUtils.class, 
locale);
         @SuppressWarnings("unchecked") // clone will produce correct type
-        Iterator<LocaleChangeListener> iterator = 
((Vector<LocaleChangeListener>) localeChangeListeners.clone()).iterator();
-
-        while (iterator.hasNext()) {
-            LocaleChangeListener listener = iterator.next();
+        // TODO but why do we need to clone the list?
+        Vector<LocaleChangeListener> listeners = 
(Vector<LocaleChangeListener>) localeChangeListeners.clone();
+        for (LocaleChangeListener listener : listeners) {
             listener.localeChanged(event);
         }
     }



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

Reply via email to