You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@qpid.apache.org by ro...@apache.org on 2009/04/11 01:37:28 UTC
svn commit: r764105 -
/qpid/trunk/qpid/java/broker/src/main/java/org/apache/qpid/server/transactionlog/BaseTransactionLog.java
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-subscribe@qpid.apache.org