X-Git-Url: http://gitweb.fperrin.net/?a=blobdiff_plain;f=tim%2Fprune%2FUpdateMessageBroker.java;h=16599735a80fc831ede28f72241f1310f42a3bb7;hb=ff33ebba6b7c62834f6dae16ce33eb2c710b160e;hp=0b21f52d0a381de1c6a293717393792fadc7ea62;hpb=ca9bdb3916f9c39adbbf95d06ac95c21dafbb4e6;p=GpsPrune.git

diff --git a/tim/prune/UpdateMessageBroker.java b/tim/prune/UpdateMessageBroker.java
index 0b21f52..1659973 100644
--- a/tim/prune/UpdateMessageBroker.java
+++ b/tim/prune/UpdateMessageBroker.java
@@ -6,9 +6,13 @@ package tim.prune;
  */
 public abstract class UpdateMessageBroker
 {
-	private static final int MAXIMUM_NUMBER_SUBSCRIBERS = 6;
+	private static final int MAXIMUM_NUMBER_SUBSCRIBERS = 8;
+	/** Array of all subscribers */
 	private static DataSubscriber[] _subscribers = new DataSubscriber[MAXIMUM_NUMBER_SUBSCRIBERS];
-	private static int _subscriberNum = 0;
+	/** Index from which to start looking for an empty slot*/
+	private static int _searchStartIndex = 0;
+	/** Enable/disabled flag */
+	private static boolean _enabled = true;
 
 
 	/**
@@ -17,10 +21,43 @@ public abstract class UpdateMessageBroker
 	 */
 	public static void addSubscriber(DataSubscriber inSub)
 	{
-		_subscribers[_subscriberNum] = inSub;
-		_subscriberNum++;
+		// Loop looking for first null entry
+		for (int i=_searchStartIndex; i<MAXIMUM_NUMBER_SUBSCRIBERS; i++)
+		{
+			if (_subscribers[i] == null)
+			{
+				_subscribers[i] = inSub;
+				_searchStartIndex = i+1;
+				break;
+			}
+		}
 	}
 
+	/**
+	 * Remove the given subscriber from the list
+	 * @param inSub subscriber to remove
+	 */
+	public static void removeSubscriber(DataSubscriber inSub)
+	{
+		for (int i=0; i<MAXIMUM_NUMBER_SUBSCRIBERS; i++)
+		{
+			if (_subscribers[i] == inSub)
+			{
+				_subscribers[i] = null;
+				// Could break out of the loop here but we want to make sure we remove all of them
+			}
+		}
+		_searchStartIndex = 0; // for the next add, start from beginning to ensure all gaps are filled
+	}
+
+	/**
+	 * Enable or disable the messaging (to allow temporary disabling for multiple operations)
+	 * @param inEnabled false to disable, true to enable again
+	 */
+	public static void enableMessaging(boolean inEnabled)
+	{
+		_enabled = inEnabled;
+	}
 
 	/**
 	 * Send a message to all subscribers that
@@ -38,6 +75,8 @@ public abstract class UpdateMessageBroker
 	 */
 	public static void informSubscribers(byte inChange)
 	{
+		// TODO: Launch separate thread so that whatever caused the inform can finish
+		if (!_enabled) return;
 		for (int i=0; i<_subscribers.length; i++)
 		{
 			if (_subscribers[i] != null)
@@ -53,6 +92,7 @@ public abstract class UpdateMessageBroker
 	 */
 	public static void informSubscribers(String inMessage)
 	{
+		if (!_enabled) return;
 		for (int i=0; i<_subscribers.length; i++)
 		{
 			if (_subscribers[i] != null)