suspendmanager: don't leave one tile unsuspended with closed down dead-ends

plule · plule · commit e867251cf56f · 2024-05-06T07:16:55.000+02:00
diff --git a/docs/changelog.txt b/docs/changelog.txt
@@ -64,6 +64,7 @@ Template for new versions:
 - `dig`: don't affect already-revealed tiles when marking z-level for warm/damp dig
 - `zone`: refresh values in distance column when switching selected pastures when the assign animals dialog is open
 - `logistics`: also add trainers to semi-wild pets when autoretrain is enabled
+- `suspendmanager`: fully suspend unbuildable deadends
 
 ## Misc Improvements
 - `caravan`: display who is in the cages you are selecting for trade and whether they are hostile
diff --git a/plugins/suspendmanager.cpp b/plugins/suspendmanager.cpp
@@ -556,7 +556,13 @@ class SuspendManager {
                 exit = impassiblePlan;
             }
 
-            if (!exit) return;
+            if (!exit) {
+                // there is no exit at all
+                // suspend the current construction job to leave the entire plan suspended
+                // and stop here
+                suspensions[job->id] = Reason::DEADEND;
+                return;
+            }
 
             // exit is the single exit point of this corridor, suspend its construction job...
             for (auto exit_job : exit->jobs) {
