fix bug

'stopCurrentWorking' may cause new calculation be cancelled, so create a
new method 'fireCalculationFailed' to avoid that problem.

Author: trytocatch

src/trytocatch/regexreplacer/model/RegexController.java

@@ -183,7 +183,7 @@ protected void calculateResult() {
 		try {
 			workOutResultImpl();
 		} catch (Throwable t) {
-			stopCurrentWorking();
+			fireCalculationFailed();
 			if (resultObserver != null)
 				resultObserver.onResultChanged(
 						null,

src/trytocatch/utils/concurrent/LatestResultsProvider.java

@@ -44,9 +44,11 @@
  * <p>
  * How to use it:<br>
  * Implement the abstract method 'calculateResult' to do your calculation with your data,
- * cancel your calculation if you detect that the method 'isWorking' returns false, call
- * the method 'updateParametersVersion' once the data has changed, call the method
- * 'update' or 'updateAndWait' to launch the calculation if you want to get the result.
+ * call 'fireCalculationFailed' while calculation failed and no Throwalbe be thrown by 
+ * 'calculateResult', cancel your calculation if you detect that the method 'isWorking'
+ * returns false, call the method 'updateParametersVersion' once the data has changed,
+ * call the method 'update' or 'updateAndWait' to launch the calculation if you want to
+ * get the result.
  * 
  * @author trytocatch@163.com
  * @date 2013-2-2
@@ -147,7 +149,7 @@ public void run() {
 									barrier.nextCycle(workingVersion);
 							} catch (Throwable t) {
 								t.printStackTrace();
-								workState.set(WS_CANCELED);
+								fireCalculationFailed();
 							}
 						}
 					} catch (InterruptedException e) {
@@ -184,10 +186,6 @@ public int getDelayUpperLimit() {
 	public void setDelayUpperLimit(int delayUpperLimit) {
 		this.delayUpperLimit = delayUpperLimit;
 	}
-	
-	public final void stopCurrentWorking() {
-		workState.set(WS_CANCELED);
-	}
 
 	/**
 	 * 
@@ -251,8 +249,22 @@ protected final void updateParametersVersion() {
 				workState.compareAndSet(WS_WORKING, WS_NEW_TASK);
 	}
 
+	public final void stopCurrentWorking() {
+		workState.set(WS_CANCELED);
+	}
+	
 	/**
 	 * implement this to deal with you task
 	 */
 	protected abstract void calculateResult();
+	
+	/**
+	 * If 'calculate the result' failed and no Throwable be thrown by
+	 * 'calculateResult', calls this to let it know so that isResultUptodate()
+	 * won't become true. If a Throwable be thrown by 'calculateResult', this
+	 * tool will catch it and call 'fireCalculationFailed'.
+	 */
+	public final void fireCalculationFailed() {
+		workState.compareAndSet(WS_WORKING, WS_CANCELED);
+	}
 }