jmeter-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From s...@apache.org
Subject svn commit: r1624276 - /jmeter/trunk/src/core/org/apache/jmeter/util/JMeterUtils.java
Date Thu, 11 Sep 2014 13:06:35 GMT
Author: sebb
Date: Thu Sep 11 13:06:34 2014
New Revision: 1624276

URL: http://svn.apache.org/r1624276
Log:
Document TODO

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

Modified: jmeter/trunk/src/core/org/apache/jmeter/util/JMeterUtils.java
URL: http://svn.apache.org/viewvc/jmeter/trunk/src/core/org/apache/jmeter/util/JMeterUtils.java?rev=1624276&r1=1624275&r2=1624276&view=diff
==============================================================================
--- jmeter/trunk/src/core/org/apache/jmeter/util/JMeterUtils.java (original)
+++ jmeter/trunk/src/core/org/apache/jmeter/util/JMeterUtils.java Thu Sep 11 13:06:34 2014
@@ -414,6 +414,8 @@ public class JMeterUtils implements Unit
         LocaleChangeEvent event = new LocaleChangeEvent(JMeterUtils.class, locale);
         @SuppressWarnings("unchecked") // clone will produce correct type
         // TODO but why do we need to clone the list?
+        // ANS: to avoid possible ConcurrentUpdateException when unsubscribing
+        // Could perhaps avoid need to clone by using a modern concurrent list
         Vector<LocaleChangeListener> listeners = (Vector<LocaleChangeListener>)
localeChangeListeners.clone();
         for (LocaleChangeListener listener : listeners) {
             listener.localeChanged(event);



Mime
View raw message