You are viewing a plain text version of this content. The canonical link for it is here.
Posted to server-dev@james.apache.org by no...@apache.org on 2010/09/20 20:48:44 UTC
svn commit: r999051 -
/james/imap/trunk/store/src/main/java/org/apache/james/mailbox/store/DelegatingMailboxListener.java
Author: norman
Date: Mon Sep 20 18:48:44 2010
New Revision: 999051
URL: http://svn.apache.org/viewvc?rev=999051&view=rev
Log:
Make sure that DelegatingMailboxListener does not loose any events when a MailboxListener was closed (IMAP-216). Thx to François-Denis Gonthier for the fix!
Modified:
james/imap/trunk/store/src/main/java/org/apache/james/mailbox/store/DelegatingMailboxListener.java
Modified: james/imap/trunk/store/src/main/java/org/apache/james/mailbox/store/DelegatingMailboxListener.java
URL: http://svn.apache.org/viewvc/james/imap/trunk/store/src/main/java/org/apache/james/mailbox/store/DelegatingMailboxListener.java?rev=999051&r1=999050&r2=999051&view=diff
==============================================================================
--- james/imap/trunk/store/src/main/java/org/apache/james/mailbox/store/DelegatingMailboxListener.java (original)
+++ james/imap/trunk/store/src/main/java/org/apache/james/mailbox/store/DelegatingMailboxListener.java Mon Sep 20 18:48:44 2010
@@ -54,7 +54,8 @@ public class DelegatingMailboxListener i
public void event(Event event) {
List<MailboxListener> mListeners = listeners.get(event.getMailboxPath());
if (mListeners != null && mListeners.isEmpty() == false) {
- for (int i = 0; i < mListeners.size(); i++) {
+ int sz = mListeners.size();
+ for (int i = 0; i < sz; i++) {
MailboxListener l = mListeners.get(i);
if (l.isClosed()) {
mListeners.remove(l);
---------------------------------------------------------------------
To unsubscribe, e-mail: server-dev-unsubscribe@james.apache.org
For additional commands, e-mail: server-dev-help@james.apache.org