Don't include raiding units in Units::isActive; make getPosition return invalid coords

docs/changelog.txt

@@ -56,6 +56,9 @@ Template for new versions:
 ## New Features
 
 ## Fixes
+- ``Units::getPosition``, ``Items::getPosition``: return invalid coord for inactive units, items held by units off map
+- ``Units::isUnitInBox``, ``Units::getUnitsInBox``: don't include inactive units
+- ``Units::isActive``: return false for units off map
 
 ## Misc Improvements
 - DFHack text edit fields now delete the character at the cursor when you hit the Delete key

docs/dev/Lua API.rst

@@ -1643,7 +1643,7 @@ Units module
 
   Returns the true *x,y,z* of the unit, or *nil* if invalid. You should
   generally use this method instead of reading *unit.pos* directly since
-  that field can be inaccurate when the unit is caged.
+  that field can be inaccurate when the unit is caged, off map, or dead.
 
 * ``dfhack.units.teleport(unit, pos)``
 

library/RemoteTools.cpp

@@ -584,7 +584,7 @@ static command_result ListUnits(color_ostream &stream,
 
     if (in->scan_all())
     {
-        auto &vec = df::unit::get_vector();
+        auto &vec = df::global::world->units.active;
 
         for (size_t i = 0; i < vec.size(); i++)
         {

library/include/modules/Items.h

@@ -137,7 +137,7 @@ DFHACK_EXPORT df::building *getHolderBuilding(df::item *item);
 // Get unit that holds the item or NULL.
 DFHACK_EXPORT df::unit *getHolderUnit(df::item *item);
 
-// Returns the true position of the item (non-trivial if in inventory).
+// Get the true position of the item (non-trivial if in inventory). Return invalid coord if holder off map.
 DFHACK_EXPORT df::coord getPosition(df::item *item);
 
 /// Returns the title of a codex or "tool", either as the codex title or as the title of the

library/include/modules/Units.h

@@ -202,7 +202,7 @@ inline auto citizensRange(std::vector<df::unit *> &vec, bool exclude_residents =
 DFHACK_EXPORT void forCitizens(std::function<void(df::unit *)> fn, bool exclude_residents = false, bool include_insane = false);
 DFHACK_EXPORT bool getCitizens(std::vector<df::unit *> &citizens, bool exclude_residents = false, bool include_insane = false);
 
-// Returns the true position of the unit (non-trivial in case of caged).
+// Get the true position of the unit (non-trivial in case of caged or inactive). Returns invalid coord if dead or off map.
 DFHACK_EXPORT df::coord getPosition(df::unit *unit);
 
 // Moves unit and any riders to the target coordinates. Sets tile occupancy flags.

library/modules/Units.cpp

@@ -122,7 +122,11 @@ constexpr uint32_t exclude_flags2 = (
 
 bool Units::isActive(df::unit *unit) {
     CHECK_NULL_POINTER(unit);
-    return !unit->flags1.bits.inactive;
+    if (unit->flags1.bits.inactive)
+        return false;
+    else if (unit->flags1.bits.move_state || unit->flags1.bits.can_swap)
+        return true; // These are always unset when leaving the map
+    return linear_index(world->units.active, &df::unit::id, unit->id) >= 0;
 }
 
 bool Units::isVisible(df::unit *unit) {
@@ -658,7 +662,7 @@ bool Units::isGreatDanger(df::unit *unit) {
 
 bool Units::isUnitInBox(df::unit *u, const cuboid &box) {
     CHECK_NULL_POINTER(u);
-    return box.containsPos(getPosition(u));
+    return isActive(u) ? box.containsPos(getPosition(u)) : false;
 }
 
 bool Units::getUnitsInBox(vector<df::unit *> &units, const cuboid &box, std::function<bool(df::unit *)> filter) {
@@ -743,6 +747,8 @@ bool Units::getCitizens(vector<df::unit *> &citizens, bool exclude_residents, bo
 
 df::coord Units::getPosition(df::unit *unit) {
     CHECK_NULL_POINTER(unit);
+    if (!isActive(unit))
+        return df::coord();
     if (unit->flags1.bits.caged) {
         if (auto cage = getContainer(unit))
             return Items::getPosition(cage);