Author: ritchiem
Date: Sat Apr 11 02:00:33 2009
New Revision: 764150

URL: http://svn.apache.org/viewvc?rev=764150&view=rev
Log:
QPID-1800: remove unecessary addAll() command, as the list is about to be 
cloned we can simply use the original list
merged from trunk r764105

Modified:
    
qpid/branches/0.5-release/qpid/java/broker/src/main/java/org/apache/qpid/server/transactionlog/BaseTransactionLog.java

Modified: 
qpid/branches/0.5-release/qpid/java/broker/src/main/java/org/apache/qpid/server/transactionlog/BaseTransactionLog.java
URL: 
http://svn.apache.org/viewvc/qpid/branches/0.5-release/qpid/java/broker/src/main/java/org/apache/qpid/server/transactionlog/BaseTransactionLog.java?rev=764150&r1=764149&r2=764150&view=diff
==============================================================================
--- 
qpid/branches/0.5-release/qpid/java/broker/src/main/java/org/apache/qpid/server/transactionlog/BaseTransactionLog.java
 (original)
+++ 
qpid/branches/0.5-release/qpid/java/broker/src/main/java/org/apache/qpid/server/transactionlog/BaseTransactionLog.java
 Sat Apr 11 02:00:33 2009
@@ -98,7 +98,7 @@
             else
             {
                 //No existing list, add all provided queues (cloning 
toEnqueueList in case someone else changes original).
-                toEnqueueList.addAll(queues);
+                toEnqueueList = queues;
                 _idToQueues.put(messageId, 
Collections.synchronizedList((ArrayList<AMQQueue>)toEnqueueList.clone()));
             }
 



---------------------------------------------------------------------
Apache Qpid - AMQP Messaging Implementation
Project:      http://qpid.apache.org
Use/Interact: mailto:[email protected]

Reply via email to