You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@jmeter.apache.org by pm...@apache.org on 2016/11/12 20:23:06 UTC

svn commit: r1769404 - /jmeter/trunk/src/components/org/apache/jmeter/control/ThroughputController.java

Author: pmouawad
Date: Sat Nov 12 20:23:05 2016
New Revision: 1769404

URL: http://svn.apache.org/viewvc?rev=1769404&view=rev
Log:
Ignore false positive on Synchronization on non final field

Modified:
    jmeter/trunk/src/components/org/apache/jmeter/control/ThroughputController.java

Modified: jmeter/trunk/src/components/org/apache/jmeter/control/ThroughputController.java
URL: http://svn.apache.org/viewvc/jmeter/trunk/src/components/org/apache/jmeter/control/ThroughputController.java?rev=1769404&r1=1769403&r2=1769404&view=diff
==============================================================================
--- jmeter/trunk/src/components/org/apache/jmeter/control/ThroughputController.java (original)
+++ jmeter/trunk/src/components/org/apache/jmeter/control/ThroughputController.java Sat Nov 12 20:23:05 2016
@@ -172,7 +172,8 @@ public class ThroughputController extend
         }
         return retVal;
     }
-
+    
+    @SuppressWarnings("SynchronizeOnNonFinalField")
     private int getExecutions() {
         if (!isPerThread()) {
             synchronized (counterLock) {
@@ -232,6 +233,7 @@ public class ThroughputController extend
     }
 
     @Override
+    @SuppressWarnings("SynchronizeOnNonFinalField")
     public void iterationStart(LoopIterationEvent iterEvent) {
         if (!isPerThread()) {
             synchronized (counterLock) {
@@ -251,6 +253,7 @@ public class ThroughputController extend
     }
 
     @Override
+    @SuppressWarnings("SynchronizeOnNonFinalField")
     public void testStarted() {
         synchronized (counterLock) {
             globalNumExecutions = new MutableInteger(0);