jmeter-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From pmoua...@apache.org
Subject svn commit: r1546654 - in /jmeter/trunk: src/components/org/apache/jmeter/visualizers/RespTimeGraphVisualizer.java xdocs/changes.xml
Date Fri, 29 Nov 2013 22:29:00 GMT
Author: pmouawad
Date: Fri Nov 29 22:29:00 2013
New Revision: 1546654

URL: http://svn.apache.org/r1546654
Log:
Bug 55826 - Unsynchronised concurrent accesses to list in field RespTimeGraphVisualizer.internalList
Bugzilla Id: 55826

Modified:
    jmeter/trunk/src/components/org/apache/jmeter/visualizers/RespTimeGraphVisualizer.java
    jmeter/trunk/xdocs/changes.xml

Modified: jmeter/trunk/src/components/org/apache/jmeter/visualizers/RespTimeGraphVisualizer.java
URL: http://svn.apache.org/viewvc/jmeter/trunk/src/components/org/apache/jmeter/visualizers/RespTimeGraphVisualizer.java?rev=1546654&r1=1546653&r2=1546654&view=diff
==============================================================================
--- jmeter/trunk/src/components/org/apache/jmeter/visualizers/RespTimeGraphVisualizer.java
(original)
+++ jmeter/trunk/src/components/org/apache/jmeter/visualizers/RespTimeGraphVisualizer.java
Fri Nov 29 22:29:00 2013
@@ -489,8 +489,8 @@ public class RespTimeGraphVisualizer ext
                 filePanel.actionPerformed(event);
             } else {
                 // Reload data form internal list of results
-                if (internalList.size() >= 2) {
-                    synchronized (lockInterval) {
+                synchronized (lockInterval) {
+                    if (internalList.size() >= 2) {
                         List<RespTimeGraphDataBean> tempList = new ArrayList<RespTimeGraphDataBean>();
                         tempList.addAll(internalList);
                         this.clearData();

Modified: jmeter/trunk/xdocs/changes.xml
URL: http://svn.apache.org/viewvc/jmeter/trunk/xdocs/changes.xml?rev=1546654&r1=1546653&r2=1546654&view=diff
==============================================================================
--- jmeter/trunk/xdocs/changes.xml (original)
+++ jmeter/trunk/xdocs/changes.xml Fri Nov 29 22:29:00 2013
@@ -144,6 +144,7 @@ A workaround is to use a Java 7 update 4
 
 <h3>Listeners</h3>
 <ul>
+<li><bugzilla>55826</bugzilla> - Unsynchronised concurrent accesses to
list in field RespTimeGraphVisualizer.internalList </li>
 </ul>
 
 <h3>Timers, Assertions, Config, Pre- &amp; Post-Processors</h3>



Mime
View raw message