X-Git-Url: https://git.pterodactylus.net/?a=blobdiff_plain;f=src%2Fnet%2Fpterodactylus%2Ffcp%2Fhighlevel%2FHighLevelCallback.java;h=541e502f030c0ff947524fc6d790133f870353cd;hb=7fd5e756aaed4ddea02db3d42e49feed86c1b839;hp=3e3e70f539118db01d5db7e59ea3981f899d9644;hpb=fa94fcde43d1c87745bcdbf25b4fa6242d5b1834;p=jFCPlib.git

diff --git a/src/net/pterodactylus/fcp/highlevel/HighLevelCallback.java b/src/net/pterodactylus/fcp/highlevel/HighLevelCallback.java
index 3e3e70f..541e502 100644
--- a/src/net/pterodactylus/fcp/highlevel/HighLevelCallback.java
+++ b/src/net/pterodactylus/fcp/highlevel/HighLevelCallback.java
@@ -35,21 +35,25 @@ import java.util.List;
 public class HighLevelCallback<R extends HighLevelResult> {
 
 	/** Object used for synchronization. */
-	private final Object syncObject = new Object();
+	protected final Object syncObject = new Object();
 
 	/** The list of callback listeners. */
 	private final List<HighLevelCallbackListener<R>> highLevelCallbackListeners = Collections.synchronizedList(new ArrayList<HighLevelCallbackListener<R>>());
 
 	/** Whether the result is complete. */
-	private boolean resultComplete = false;
+	protected boolean resultComplete = false;
 
 	/** The result of the operation. */
 	private R result = null;
 
 	/**
 	 * Package-private construtor.
+	 * 
+	 * @param result
+	 *            The result of the operation
 	 */
-	HighLevelCallback() {
+	HighLevelCallback(R result) {
+		this.result = result;
 	}
 
 	/**
@@ -80,15 +84,17 @@ public class HighLevelCallback<R extends HighLevelResult> {
 	}
 
 	/**
-	 * Notifies all listeners that the result of the operation is now known.
+	 * Notifies all listeners that the result of the operation is now known. As
+	 * soon as a listener was notified it will be removed from the list of
+	 * listeners!
 	 * 
 	 * @see HighLevelCallbackListener#gotResult(HighLevelCallback)
 	 */
 	private synchronized void fireGotResult() {
 		for (HighLevelCallbackListener<R> highLevelCallbackListener: highLevelCallbackListeners) {
-			highLevelCallbackListeners.remove(highLevelCallbackListener);
 			highLevelCallbackListener.gotResult(this);
 		}
+		highLevelCallbackListeners.clear();
 	}
 
 	/**
@@ -137,44 +143,6 @@ public class HighLevelCallback<R extends HighLevelResult> {
 	}
 
 	/**
-	 * Sets the complete result of the operation. Calling this method will
-	 * result in all listeners being notified.
-	 * 
-	 * @see #fireGotResult()
-	 * @param result
-	 *            The result of the operation
-	 */
-	void setResult(R result) {
-		setResult(result, true);
-	}
-
-	/**
-	 * Sets the result of the operation. Depending on the <code>notify</code>
-	 * parameter the listeners are notified. You have to call this method with
-	 * <code>notify = true</code> after your result is completed, otherwise
-	 * clients will block endlessly!
-	 * 
-	 * @param result
-	 *            The result of the operation
-	 * @param notify
-	 *            <code>true</code> to finalize the result and notify all
-	 *            listeners, <code>false</code> if something in the result
-	 *            might still change
-	 */
-	void setResult(R result, boolean notify) {
-		synchronized (syncObject) {
-			this.result = result;
-			if (notify) {
-				resultComplete = true;
-				syncObject.notifyAll();
-			}
-		}
-		if (notify) {
-			fireGotResult();
-		}
-	}
-
-	/**
 	 * Returns the result even if it is not yet complete.
 	 * 
 	 * @return The result of the operation
@@ -186,9 +154,8 @@ public class HighLevelCallback<R extends HighLevelResult> {
 	}
 
 	/**
-	 * Marks the result given in with
-	 * {@link #setResult(HighLevelResult, boolean)} as complete and notify the
-	 * listeners. If the result was already complete, nothing will be done.
+	 * Marks the result as complete and notify the listeners. If the result was
+	 * already complete, nothing will be done.
 	 */
 	void setDone() {
 		synchronized (syncObject) {