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