From 647897cd6d097986d42d3f1822e2da435f8dddb8 Mon Sep 17 00:00:00 2001
From: Dave Hoover <dhoover@threerings.net>
Date: Wed, 10 Jun 2009 21:17:51 +0000
Subject: [PATCH] This may cause small breakage for folks extending
 ChatDirector, but let's make these things that just add & remove from our
 collections of things return the values from their collections.

git-svn-id: svn+ssh://src.earth.threerings.net/narya/trunk@5823 542714f4-19e9-0310-aa3c-eee0fc999fb1
---
 .../crowd/chat/client/ChatDirector.java       | 26 ++++++++++---------
 1 file changed, 14 insertions(+), 12 deletions(-)

diff --git a/src/java/com/threerings/crowd/chat/client/ChatDirector.java b/src/java/com/threerings/crowd/chat/client/ChatDirector.java
index 71069df30..7fff94d5a 100644
--- a/src/java/com/threerings/crowd/chat/client/ChatDirector.java
+++ b/src/java/com/threerings/crowd/chat/client/ChatDirector.java
@@ -184,52 +184,54 @@ public class ChatDirector extends BasicDirector
      * Adds the supplied chat display to the end of the chat display list. It will subsequently be
      * notified of incoming chat messages as well as tell responses.
      */
-    public void addChatDisplay (ChatDisplay display)
+    public boolean addChatDisplay (ChatDisplay display)
     {
-        _displays.add(display);
+        return _displays.add(display);
     }
 
     /**
      * Removes the specified chat display from the chat display list. The display will no longer
      * receive chat related notifications.
      */
-    public void removeChatDisplay (ChatDisplay display)
+    public boolean removeChatDisplay (ChatDisplay display)
     {
-        _displays.remove(display);
+        return _displays.remove(display);
     }
 
     /**
      * Adds the specified chat filter to the list of filters. All chat requests and receipts will
      * be filtered with all filters before they being sent or dispatched locally.
      */
-    public void addChatFilter (ChatFilter filter)
+    public boolean addChatFilter (ChatFilter filter)
     {
-        _filters.add(filter);
+        return _filters.add(filter);
     }
 
     /**
      * Removes the specified chat filter from the list of chat filter.
      */
-    public void removeChatFilter (ChatFilter filter)
+    public boolean removeChatFilter (ChatFilter filter)
     {
-        _filters.remove(filter);
+        return _filters.remove(filter);
     }
 
     /**
      * Adds an observer that watches the chatters list, and updates it immediately.
      */
-    public void addChatterObserver (ChatterObserver co)
+    public boolean addChatterObserver (ChatterObserver co)
     {
-        _chatterObservers.add(co);
+        boolean added = _chatterObservers.add(co);
         co.chattersUpdated(_chatters.listIterator());
+
+        return added;
     }
 
     /**
      * Removes an observer from the list of chatter observers.
      */
-    public void removeChatterObserver (ChatterObserver co)
+    public boolean removeChatterObserver (ChatterObserver co)
     {
-        _chatterObservers.remove(co);
+        return _chatterObservers.remove(co);
     }
 
     /**