Author: sebb
Date: Fri Oct 29 17:03:25 2010
New Revision: 1028846

URL: http://svn.apache.org/viewvc?rev=1028846&view=rev
Log:
Don't use == to compare strings; use constant for "true".

Modified:
    jakarta/jmeter/trunk/src/core/org/apache/jmeter/threads/JMeterThread.java

Modified: 
jakarta/jmeter/trunk/src/core/org/apache/jmeter/threads/JMeterThread.java
URL: 
http://svn.apache.org/viewvc/jakarta/jmeter/trunk/src/core/org/apache/jmeter/threads/JMeterThread.java?rev=1028846&r1=1028845&r2=1028846&view=diff
==============================================================================
--- jakarta/jmeter/trunk/src/core/org/apache/jmeter/threads/JMeterThread.java 
(original)
+++ jakarta/jmeter/trunk/src/core/org/apache/jmeter/threads/JMeterThread.java 
Fri Oct 29 17:03:25 2010
@@ -68,6 +68,8 @@ public class JMeterThread implements Run
 
     public static final String LAST_SAMPLE_OK = "JMeterThread.last_sample_ok"; 
// $NON-NLS-1$
 
+    private static final String TRUE = Boolean.toString(true); // i.e. "true"
+
     private final Controller controller;
 
     private final HashTree testTree;
@@ -246,9 +248,9 @@ public class JMeterThread implements Run
                     if (onErrorStartNextLoop) { // if the threadGroup option 
is to start next loop when it fails
 
                         if (sam.equals(firstSampler)) { // if it's the start 
of an iteration
-                            threadContext.getVariables().put(LAST_SAMPLE_OK, 
"true");
+                            threadContext.getVariables().put(LAST_SAMPLE_OK, 
TRUE);
                         }
-                        if (threadContext.getVariables().get(LAST_SAMPLE_OK) 
== "true") {
+                        if 
(TRUE.equals(threadContext.getVariables().get(LAST_SAMPLE_OK))) {
                             process_sampler(sam, null, threadContext);
                             sam = controller.next();
                         } else {
@@ -461,7 +463,7 @@ public class JMeterThread implements Run
     private void initRun(JMeterContext threadContext) {
         threadContext.setVariables(threadVars);
         threadContext.setThreadNum(getThreadNum());
-        threadContext.getVariables().put(LAST_SAMPLE_OK, "true");
+        threadContext.getVariables().put(LAST_SAMPLE_OK, TRUE);
         threadContext.setThread(this);
         threadContext.setThreadGroup(threadGroup);
         threadContext.setEngine(engine);



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

Reply via email to