You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@qpid.apache.org by rg...@apache.org on 2016/11/15 12:14:58 UTC
svn commit: r1769796 -
/qpid/java/branches/remove-queue-runner/broker-core/src/main/java/org/apache/qpid/server/queue/QueueConsumerManagerImpl.java
Author: rgodfrey
Date: Tue Nov 15 12:14:58 2016
New Revision: 1769796
URL: http://svn.apache.org/viewvc?rev=1769796&view=rev
Log:
Iterators over the interested list may throw a NoSuchElementException even when hasNext() had previously been called
Modified:
qpid/java/branches/remove-queue-runner/broker-core/src/main/java/org/apache/qpid/server/queue/QueueConsumerManagerImpl.java
Modified: qpid/java/branches/remove-queue-runner/broker-core/src/main/java/org/apache/qpid/server/queue/QueueConsumerManagerImpl.java
URL: http://svn.apache.org/viewvc/qpid/java/branches/remove-queue-runner/broker-core/src/main/java/org/apache/qpid/server/queue/QueueConsumerManagerImpl.java?rev=1769796&r1=1769795&r2=1769796&view=diff
==============================================================================
--- qpid/java/branches/remove-queue-runner/broker-core/src/main/java/org/apache/qpid/server/queue/QueueConsumerManagerImpl.java (original)
+++ qpid/java/branches/remove-queue-runner/broker-core/src/main/java/org/apache/qpid/server/queue/QueueConsumerManagerImpl.java Tue Nov 15 12:14:58 2016
@@ -263,7 +263,8 @@ public class QueueConsumerManagerImpl im
@Override
public QueueConsumer<?> next()
{
- return _underlying.next().getQueueConsumer();
+ QueueConsumerNode next = _underlying.next();
+ return next == null ? null : next.getQueueConsumer();
}
@Override
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@qpid.apache.org
For additional commands, e-mail: commits-help@qpid.apache.org