fix autochop report bug

fix format string specification in `autochop`
fixes #5701

Author: ab9rf

docs/changelog.txt

@@ -59,14 +59,17 @@ Template for new versions:
 ## New Features
 
 ## Fixes
+- `autochop`: the report will no longer throw a C++ exception when burrows are defined.
 
 ## Misc Improvements
 
 ## Documentation
 
 ## API
+- Added ``Burrows::getName``: obtains the name of a burrow, or the same placeholder name that DF would show if the burrow is unnamed.
 
 ## Lua
+- Added ``Burrows::getName`` as ``dfhack.burrows.getName``.
 
 ## Removed
 

docs/dev/Lua API.rst

@@ -2489,6 +2489,12 @@ Maps module
 Burrows module
 --------------
 
+* ``dfhack.burrows.getName(burrow)``
+
+  Returns the name of the burrow.
+  If the burrow has no set name, returns the same placeholder name that DF would show in the UI.
+
+  Returns a table of all burrow pointers.
 * ``dfhack.burrows.findByName(name[, ignore_final_plus])``
 
   Returns the burrow pointer or *nil*. if ``ignore_final_plus`` is ``true``,

plugins/autochop.cpp

@@ -669,9 +669,8 @@ static void autochop_printStatus(color_ostream &out) {
     for (auto &burrow : plotinfo->burrows.list) {
         name_width = std::max(name_width, (int)burrow->name.size());
     }
-    name_width = -name_width; // left justify
 
-    constexpr auto fmt = "{:{}}  {:4}  {:4}  {:8}  {:5}  {:6}  {:7}\n";
+    constexpr auto fmt = "{:<{}}  {:4}  {:4}  {:8}  {:5}  {:6}  {:7}\n";
     out.print(fmt, "burrow name", name_width, " id ", "chop", "clearcut", "trees", "marked", "protect");
     out.print(fmt, "-----------", name_width, "----", "----", "--------", "-----", "------", "-------");
 
@@ -681,6 +680,7 @@ static void autochop_printStatus(color_ostream &out) {
         bool protect_brewable = false;
         bool protect_edible = false;
         bool protect_cookable = false;
+        std::string name = burrow->name.empty() ? fmt::format("Burrow {}", burrow->id+1) : burrow->name;
         if (watched_burrows_indices.count(burrow->id)) {
             auto &c = watched_burrows[watched_burrows_indices[burrow->id]];
             chop = c.get_bool(BURROW_CONFIG_CHOP);
@@ -689,7 +689,7 @@ static void autochop_printStatus(color_ostream &out) {
             protect_edible = c.get_bool(BURROW_CONFIG_PROTECT_EDIBLE);
             protect_cookable = c.get_bool(BURROW_CONFIG_PROTECT_COOKABLE);
         }
-        out.print(fmt, burrow->name, name_width, burrow->id,
+        out.print(fmt, name, name_width, burrow->id,
                 chop ? "[x]" : "[ ]", clearcut ? "[x]" : "[ ]",
                 tree_counts[burrow->id],
                 designated_tree_counts[burrow->id],