diff options
author | Gordon Sim <gsim@apache.org> | 2012-01-10 18:52:28 +0000 |
---|---|---|
committer | Gordon Sim <gsim@apache.org> | 2012-01-10 18:52:28 +0000 |
commit | 2e6a4681e48bd55926150690c5403f8173c21506 (patch) | |
tree | 6f7e3aa85dc476fedaa14f47802fdd66cf3425db | |
parent | 698dffce843f2e7c049607488f29373ce40c2713 (diff) | |
download | qpid-python-2e6a4681e48bd55926150690c5403f8173c21506.tar.gz |
QPID-3603: Ensure listeners are notified of available messages even when a consumer also exists (note that replicating browser sees acquired messages)
git-svn-id: https://svn.apache.org/repos/asf/qpid/branches/qpid-3603@1229681 13f79535-47bb-0310-9956-ffa450edef68
-rw-r--r-- | qpid/cpp/src/qpid/broker/QueueListeners.cpp | 13 |
1 files changed, 6 insertions, 7 deletions
diff --git a/qpid/cpp/src/qpid/broker/QueueListeners.cpp b/qpid/cpp/src/qpid/broker/QueueListeners.cpp index 591f4443bb..32c208b073 100644 --- a/qpid/cpp/src/qpid/broker/QueueListeners.cpp +++ b/qpid/cpp/src/qpid/broker/QueueListeners.cpp @@ -54,13 +54,12 @@ void QueueListeners::populate(NotificationSet& set) set.consumer = consumers.front(); consumers.pop_front(); set.consumer->inListeners = false; - } else { - // Don't swap the deques, hang on to the memory allocated. - set.browsers = browsers; - browsers.clear(); - for (Listeners::iterator i = set.browsers.begin(); i != set.browsers.end(); i++) - (*i)->inListeners = false; } + // Don't swap the deques, hang on to the memory allocated. + set.browsers = browsers; + browsers.clear(); + for (Listeners::iterator i = set.browsers.begin(); i != set.browsers.end(); i++) + (*i)->inListeners = false; } void QueueListeners::add(Listeners& listeners, Consumer::shared_ptr c) @@ -77,7 +76,7 @@ void QueueListeners::remove(Listeners& listeners, Consumer::shared_ptr c) void QueueListeners::NotificationSet::notify() { if (consumer) consumer->notify(); - else std::for_each(browsers.begin(), browsers.end(), boost::mem_fn(&Consumer::notify)); + std::for_each(browsers.begin(), browsers.end(), boost::mem_fn(&Consumer::notify)); } bool QueueListeners::contains(Consumer::shared_ptr c) const { |