Author: robbie
Date: Fri Apr 10 23:37:27 2009
New Revision: 764105

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

Modified:
    
qpid/trunk/qpid/java/broker/src/main/java/org/apache/qpid/server/transactionlog/BaseTransactionLog.java

Modified: 
qpid/trunk/qpid/java/broker/src/main/java/org/apache/qpid/server/transactionlog/BaseTransactionLog.java
URL: 
http://svn.apache.org/viewvc/qpid/trunk/qpid/java/broker/src/main/java/org/apache/qpid/server/transactionlog/BaseTransactionLog.java?rev=764105&r1=764104&r2=764105&view=diff
==============================================================================
--- 
qpid/trunk/qpid/java/broker/src/main/java/org/apache/qpid/server/transactionlog/BaseTransactionLog.java
 (original)
+++ 
qpid/trunk/qpid/java/broker/src/main/java/org/apache/qpid/server/transactionlog/BaseTransactionLog.java
 Fri Apr 10 23:37:27 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:commits-subscr...@qpid.apache.org

Reply via email to