You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@jmeter.apache.org by se...@apache.org on 2012/09/13 12:12:54 UTC
svn commit: r1384263 -
/jmeter/trunk/src/components/org/apache/jmeter/config/RandomVariableConfig.java
Author: sebb
Date: Thu Sep 13 10:12:54 2012
New Revision: 1384263
URL: http://svn.apache.org/viewvc?rev=1384263&view=rev
Log:
Document why methods are synchronised
Modified:
jmeter/trunk/src/components/org/apache/jmeter/config/RandomVariableConfig.java
Modified: jmeter/trunk/src/components/org/apache/jmeter/config/RandomVariableConfig.java
URL: http://svn.apache.org/viewvc/jmeter/trunk/src/components/org/apache/jmeter/config/RandomVariableConfig.java?rev=1384263&r1=1384262&r2=1384263&view=diff
==============================================================================
--- jmeter/trunk/src/components/org/apache/jmeter/config/RandomVariableConfig.java (original)
+++ jmeter/trunk/src/components/org/apache/jmeter/config/RandomVariableConfig.java Thu Sep 13 10:12:54 2012
@@ -39,6 +39,11 @@ public class RandomVariableConfig extend
private static final long serialVersionUID = 233L;
+ /*
+ * N.B. this class is shared between threads (NoThreadClone) so all access to variables
+ * needs to be protected by a lock (either sync. or volatile) to ensure safe publication.
+ */
+
private String minimumValue;
private String maximumValue;