Author: rdonkin Date: Fri Nov 16 13:50:35 2007 New Revision: 595839 URL: http://svn.apache.org/viewvc?rev=595839&view=rev Log: Using CopyOnWriteArraySet should ensure safe concurrent access. Listeners should be read often but only modified rarely so this seems a better match than synchronisation.
Modified: james/server/trunk/core-library/src/main/java/org/apache/james/mailboxmanager/impl/MailboxEventDispatcher.java Modified: james/server/trunk/core-library/src/main/java/org/apache/james/mailboxmanager/impl/MailboxEventDispatcher.java URL: http://svn.apache.org/viewvc/james/server/trunk/core-library/src/main/java/org/apache/james/mailboxmanager/impl/MailboxEventDispatcher.java?rev=595839&r1=595838&r2=595839&view=diff ============================================================================== --- james/server/trunk/core-library/src/main/java/org/apache/james/mailboxmanager/impl/MailboxEventDispatcher.java (original) +++ james/server/trunk/core-library/src/main/java/org/apache/james/mailboxmanager/impl/MailboxEventDispatcher.java Fri Nov 16 13:50:35 2007 @@ -19,16 +19,17 @@ package org.apache.james.mailboxmanager.impl; -import java.util.HashSet; import java.util.Iterator; import java.util.Set; import org.apache.james.mailboxmanager.MailboxListener; import org.apache.james.mailboxmanager.MessageResult; +import EDU.oswego.cs.dl.util.concurrent.CopyOnWriteArraySet; + public class MailboxEventDispatcher implements MailboxListener { - private Set listeners = new HashSet(); + private final Set listeners = new CopyOnWriteArraySet(); public void addMailboxListener(MailboxListener mailboxListener) { listeners.add(mailboxListener); --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]