You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@qpid.apache.org by ri...@apache.org on 2009/04/11 04:00:34 UTC

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

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:commits-subscribe@qpid.apache.org