Kill path finding jobs upon interruption, for #1873.

BuildCraft · Jun 22, 2014 · e4ab3b0 · e4ab3b0
1 parent eeb637d
commit e4ab3b0
Show file tree

Hide file tree

Showing 4 changed files with 31 additions and 1 deletion.
diff --git a/common/buildcraft/core/robots/AIRobotMoveToBlock.java b/common/buildcraft/core/robots/AIRobotMoveToBlock.java
@@ -95,4 +95,11 @@ private void setNextInPath() {
 			prevDistance = Double.MAX_VALUE;
 		}
 	}
+
+	@Override
+	public void end() {
+		if (pathSearchJob != null) {
+			pathSearchJob.terminate();
+		}
+	}
 }
diff --git a/common/buildcraft/core/robots/boards/AIRobotGoToRandomDirt.java b/common/buildcraft/core/robots/boards/AIRobotGoToRandomDirt.java
@@ -74,4 +74,10 @@ public void delegateAIEnded(AIRobot ai) {
 		}
 	}
 
+	@Override
+	public void end() {
+		if (pathFindingJob != null) {
+			pathFindingJob.terminate();
+		}
+	}
 }
diff --git a/common/buildcraft/core/robots/boards/AIRobotSearchBlock.java b/common/buildcraft/core/robots/boards/AIRobotSearchBlock.java
@@ -55,4 +55,11 @@ public void delegateAIEnded(AIRobot ai) {
 			terminate();
 		}
 	}
+
+	@Override
+	public void end() {
+		if (blockScannerJob != null) {
+			blockScannerJob.terminate();
+		}
+	}
 }
diff --git a/common/buildcraft/core/utils/PathFindingJob.java b/common/buildcraft/core/utils/PathFindingJob.java
@@ -12,16 +12,26 @@ public class PathFindingJob extends Thread {
 
 	private PathFinding pathFinding;
 
+	private boolean stop = false;
+
 	public PathFindingJob(PathFinding iPathFinding) {
 		super("Path Finding");
 		pathFinding = iPathFinding;
 	}
 
 	@Override
 	public void run() {
-		while (!pathFinding.isDone()) {
+		while (!isTerminated() && !pathFinding.isDone()) {
 			pathFinding.iterate();
 		}
 	}
 
+	public synchronized void terminate() {
+		stop = true;
+	}
+
+	public synchronized boolean isTerminated() {
+		return stop;
+	}
+
 }