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
This commit is contained in:
Dave Hoover
2009-06-10 21:17:51 +00:00
parent d85244e4e9
commit 647897cd6d
@@ -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 * 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. * 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 * Removes the specified chat display from the chat display list. The display will no longer
* receive chat related notifications. * 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 * 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. * 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. * 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. * 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()); co.chattersUpdated(_chatters.listIterator());
return added;
} }
/** /**
* Removes an observer from the list of chatter observers. * 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);
} }
/** /**