Author: sebb
Date: Tue Jul 22 08:46:58 2008
New Revision: 678779

URL: http://svn.apache.org/viewvc?rev=678779&view=rev
Log:
Make instance fields final to improve thread safety

Modified:
    
jakarta/jmeter/trunk/src/protocol/jms/org/apache/jmeter/protocol/jms/sampler/FixedQueueExecutor.java
    
jakarta/jmeter/trunk/src/protocol/jms/org/apache/jmeter/protocol/jms/sampler/TemporaryQueueExecutor.java

Modified: 
jakarta/jmeter/trunk/src/protocol/jms/org/apache/jmeter/protocol/jms/sampler/FixedQueueExecutor.java
URL: 
http://svn.apache.org/viewvc/jakarta/jmeter/trunk/src/protocol/jms/org/apache/jmeter/protocol/jms/sampler/FixedQueueExecutor.java?rev=678779&r1=678778&r2=678779&view=diff
==============================================================================
--- 
jakarta/jmeter/trunk/src/protocol/jms/org/apache/jmeter/protocol/jms/sampler/FixedQueueExecutor.java
 (original)
+++ 
jakarta/jmeter/trunk/src/protocol/jms/org/apache/jmeter/protocol/jms/sampler/FixedQueueExecutor.java
 Tue Jul 22 08:46:58 2008
@@ -27,17 +27,21 @@
 
 /**
  * Request/reply executor with a fixed reply queue. <br>
+ * 
+ * Used by JMS Sampler (Point to Point)
+ * 
  * Created on: October 28, 2004
  *
  */
 public class FixedQueueExecutor implements QueueExecutor {
+
+    private static final Logger log = LoggingManager.getLoggerForClass();
+
     /** Sender. */
-    private QueueSender producer;
+    private final QueueSender producer;
 
     /** Timeout used for waiting on message. */
-    private int timeout;
-
-    private static final Logger log = LoggingManager.getLoggerForClass();
+    private final int timeout;
 
     /**
      * Constructor.

Modified: 
jakarta/jmeter/trunk/src/protocol/jms/org/apache/jmeter/protocol/jms/sampler/TemporaryQueueExecutor.java
URL: 
http://svn.apache.org/viewvc/jakarta/jmeter/trunk/src/protocol/jms/org/apache/jmeter/protocol/jms/sampler/TemporaryQueueExecutor.java?rev=678779&r1=678778&r2=678779&view=diff
==============================================================================
--- 
jakarta/jmeter/trunk/src/protocol/jms/org/apache/jmeter/protocol/jms/sampler/TemporaryQueueExecutor.java
 (original)
+++ 
jakarta/jmeter/trunk/src/protocol/jms/org/apache/jmeter/protocol/jms/sampler/TemporaryQueueExecutor.java
 Tue Jul 22 08:46:58 2008
@@ -26,13 +26,16 @@
 
 /**
  * Request/reply executor with a temporary reply queue. <br>
+ * 
+ * Used by JMS Sampler (Point to Point)
+ * 
  * Created on: October 28, 2004
  *
  * @version $Revision$
  */
 public class TemporaryQueueExecutor implements QueueExecutor {
     /** The sender and receiver. */
-    private QueueRequestor requestor;
+    private final QueueRequestor requestor;
 
     /**
      * Constructor.



---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to