Merge branch 'develop' into myk_keybindings

docs/changelog.txt

@@ -44,12 +44,15 @@ changelog.txt uses a syntax similar to RST, with a few special sequences:
 - `autodump`: new hotkey for ``autodump-destroy-here``: Ctrl-H
 - `dig`: new hotkeys for vein designation on z-level (Ctrl-V) and vein designation across z-levels (Ctrl-Shift-V)
 - `clean`: new hotkey for `spotclean`: Ctrl-C
+- `autobutcher`: changed defaults from 5 females / 1 male to 4 females / 2 males so a single unfortunate accident doesn't leave players without a mating pair
+- `autobutcher`: now immediately loads races available at game start into the watchlist
 
 ## Documentation
 
 ## API
 
 ## Lua
+- `overlay`: overlay widgets can now specify focus paths for the viewscreens they attach to so they only appear in specific contexts. see `overlay-dev-guide` for details.
 
 ## Removed
 

docs/dev/overlay-dev-guide.rst

@@ -109,7 +109,11 @@ The ``overlay.OverlayWidget`` superclass defines the following class attributes:
     ``dwarfmode`` and the adventure mode map viewscreen would be
     ``dungeonmode``. If there is only one viewscreen that this widget is
     associated with, it can be specified as a string instead of a list of
-    strings with a single element.
+    strings with a single element. If you only want your widget to appear in
+    certain contexts, you can specify a focus path, in the same syntax as the
+    `keybinding` command. For example, ``dwarfmode/Info/CREATURES/CITIZEN`` will
+    ensure the overlay widget is only displayed when the "Citizens" subtab under
+    the "Units" panel is active.
 - ``hotspot`` (default: ``false``)
     If set to ``true``, your widget's ``overlay_onupdate`` function will be
     called whenever the `overlay` plugin's ``plugin_onupdate()`` function is

docs/plugins/autobutcher.rst

@@ -18,8 +18,8 @@ watch list. Units will be ignored if they are:
 Creatures who will not reproduce (because they're not interested in the
 opposite sex or have been gelded) will be butchered before those who will.
 Older adults and younger children will be butchered first if the population
-is above the target (defaults are: 1 male kid, 5 female kids, 1 male adult,
-5 female adults). Note that you may need to set a target above 1 to have a
+is above the target (defaults are: 2 male kids, 4 female kids, 2 male adults,
+4 female adults). Note that you may need to set a target above 1 to have a
 reliable breeding population due to asexuality etc. See `fix-ster` if this is a
 problem.
 
@@ -34,7 +34,7 @@ Usage
 ``autobutcher autowatch``
     Automatically add all new races (animals you buy from merchants, tame
     yourself, or get from migrants) to the watch list using the default target
-    counts.
+    counts. This option is enabled by default.
 ``autobutcher noautowatch``
     Stop auto-adding new races to the watch list.
 ``autobutcher target <fk> <mk> <fa> <ma> all|new|<race> [<race> ...]``
@@ -108,4 +108,3 @@ fortress::
     autobutcher target 2 2 4 2 ALPACA SHEEP LLAMA
     autobutcher target 5 5 6 2 PIG
     autobutcher target 0 0 0 0 new
-    autobutcher autowatch

library/lua/gui/widgets.lua

@@ -808,7 +808,7 @@ local function scrollbar_get_max_pos_and_height(scrollbar)
     local frame_body = scrollbar.frame_body
     local scrollbar_body_height = (frame_body and frame_body.height or 3) - 2
 
-    local height = math.max(1, math.floor(
+    local height = math.max(2, math.floor(
         (math.min(scrollbar.elems_per_page, scrollbar.num_elems) * scrollbar_body_height) /
         scrollbar.num_elems))
 

plugins/autobutcher.cpp

@@ -107,6 +107,8 @@ DFhackCExport command_result plugin_enable(color_ostream &out, bool enable) {
         DEBUG(status,out).print("%s from the API; persisting\n",
                                 is_enabled ? "enabled" : "disabled");
         set_config_bool(CONFIG_IS_ENABLED, is_enabled);
+        if (enable)
+            autobutcher_cycle(out);
     } else {
         DEBUG(status,out).print("%s from the API, but already %s; no action\n",
                                 is_enabled ? "enabled" : "disabled",
@@ -130,11 +132,11 @@ DFhackCExport command_result plugin_load_data (color_ostream &out) {
         config = World::AddPersistentData(CONFIG_KEY);
         set_config_bool(CONFIG_IS_ENABLED, is_enabled);
         set_config_val(CONFIG_CYCLE_TICKS, 6000);
-        set_config_bool(CONFIG_AUTOWATCH, false);
-        set_config_val(CONFIG_DEFAULT_FK, 5);
-        set_config_val(CONFIG_DEFAULT_MK, 1);
-        set_config_val(CONFIG_DEFAULT_FA, 5);
-        set_config_val(CONFIG_DEFAULT_MA, 1);
+        set_config_bool(CONFIG_AUTOWATCH, true);
+        set_config_val(CONFIG_DEFAULT_FK, 4);
+        set_config_val(CONFIG_DEFAULT_MK, 2);
+        set_config_val(CONFIG_DEFAULT_FA, 4);
+        set_config_val(CONFIG_DEFAULT_MA, 2);
     }
 
     // we have to copy our enabled flag into the global plugin variable, but

plugins/autochop.cpp

@@ -143,6 +143,8 @@ DFhackCExport command_result plugin_enable(color_ostream &out, bool enable) {
         DEBUG(status,out).print("%s from the API; persisting\n",
                                 is_enabled ? "enabled" : "disabled");
         set_config_bool(config, CONFIG_IS_ENABLED, is_enabled);
+        if (enable)
+            do_cycle(out, true);
     } else {
         DEBUG(status,out).print("%s from the API, but already %s; no action\n",
                                 is_enabled ? "enabled" : "disabled",

plugins/automelt.cpp

@@ -1,37 +1,21 @@
-
 #include "Debug.h"
 #include "LuaTools.h"
 #include "PluginManager.h"
-#include "TileTypes.h"
 
 #include "modules/Buildings.h"
-#include "modules/Maps.h"
 #include "modules/Items.h"
 #include "modules/World.h"
-#include "modules/Designations.h"
 #include "modules/Persistence.h"
-#include "modules/Units.h"
-#include "modules/Screen.h"
 #include "modules/Gui.h"
 
-// #include "uicommon.h"
-
 #include "df/world.h"
-#include "df/building.h"
-#include "df/world_raws.h"
-#include "df/building_def.h"
-#include "df/viewscreen_dwarfmodest.h"
 #include "df/building_stockpilest.h"
-#include "df/plotinfost.h"
 #include "df/item_quality.h"
 
 #include <map>
 #include <unordered_map>
 
-using df::building_stockpilest;
 using std::map;
-using std::multimap;
-using std::pair;
 using std::string;
 using std::unordered_map;
 using std::vector;
@@ -41,10 +25,7 @@ using namespace df::enums;
 
 DFHACK_PLUGIN("automelt");
 DFHACK_PLUGIN_IS_ENABLED(is_enabled);
-REQUIRE_GLOBAL(gps);
 REQUIRE_GLOBAL(world);
-REQUIRE_GLOBAL(cursor);
-REQUIRE_GLOBAL(plotinfo);
 
 namespace DFHack
 {
@@ -57,16 +38,12 @@ static const string CONFIG_KEY = string(plugin_name) + "/config";
 static const string STOCKPILE_CONFIG_KEY_PREFIX = string(plugin_name) + "/stockpile/";
 static PersistentDataItem config;
 
-// static vector<PersistentDataItem> watched_stockpiles;
-// static unordered_map<int, size_t> watched_stockpiles_indices;
-
 static unordered_map<int32_t, PersistentDataItem> watched_stockpiles;
 
 enum StockpileConfigValues
 {
     STOCKPILE_CONFIG_ID = 0,
     STOCKPILE_CONFIG_MONITORED = 1,
-
 };
 
 static int get_config_val(PersistentDataItem &c, int index)
@@ -115,8 +92,8 @@ static void remove_stockpile_config(color_ostream &out, int id)
     watched_stockpiles.erase(id);
 }
 
-static bool isStockpile(df::building * building) {
-    return building->getType() == df::building_type::Stockpile;
+static bool isStockpile(df::building * bld) {
+    return bld && bld->getType() == df::building_type::Stockpile;
 }
 
 static void validate_stockpile_configs(color_ostream &out)
@@ -124,7 +101,7 @@ static void validate_stockpile_configs(color_ostream &out)
     for (auto &c : watched_stockpiles) {
         int id = get_config_val(c.second, STOCKPILE_CONFIG_ID);
         auto bld = df::building::find(id);
-        if (!bld || !isStockpile(bld))
+        if (!isStockpile(bld))
             remove_stockpile_config(out, id);
     }
 }
@@ -135,7 +112,7 @@ static int32_t cycle_timestamp = 0; // world->frame_counter at last cycle
 static command_result do_command(color_ostream &out, vector<string> &parameters);
 static int32_t do_cycle(color_ostream &out);
 
-DFhackCExport command_result plugin_init(color_ostream &out, std::vector<PluginCommand> &commands)
+DFhackCExport command_result plugin_init(color_ostream &out, vector<PluginCommand> &commands)
 {
     DEBUG(status, out).print("initializing %s\n", plugin_name);
 
@@ -222,7 +199,6 @@ DFhackCExport command_result plugin_onupdate(color_ostream &out)
     return CR_OK;
 }
 
-
 static bool call_automelt_lua(color_ostream *out, const char *fn_name,
         int nargs = 0, int nres = 0,
         Lua::LuaLambda && args_lambda = Lua::DEFAULT_LUA_LAMBDA,
@@ -268,6 +244,8 @@ static command_result do_command(color_ostream &out, vector<string> &parameters)
 
 static inline bool is_metal_item(df::item *item)
 {
+    if (!item)
+        return false;
     MaterialInfo mat(item);
     return (mat.getCraftClass() == craft_material_class::Metal);
 }
@@ -307,6 +285,9 @@ static inline bool can_melt(df::item *item)
 {
     static const BadFlagsCanMelt bad_flags;
 
+    if (!is_metal_item(item))
+        return false;
+
     if (item->flags.whole & bad_flags.whole)
         return false;
 
@@ -315,9 +296,6 @@ static inline bool can_melt(df::item *item)
     if (t == df::enums::item_type::BOX || t == df::enums::item_type::BAR)
         return false;
 
-    if (!is_metal_item(item))
-        return false;
-
     for (auto &g : item->general_refs)
     {
         switch (g->getType())
@@ -372,7 +350,7 @@ static int mark_item(color_ostream &out, df::item *item, BadFlagsMarkItem bad_fl
     {
         DEBUG(perf,out).print("assignedToStockpile\n");
         size_t marked_count = 0;
-        std::vector<df::item *> contents;
+        vector<df::item *> contents;
         Items::getContainedItems(item, &contents);
         for (auto child = contents.begin(); child != contents.end(); child++)
         {
@@ -414,7 +392,6 @@ static int mark_item(color_ostream &out, df::item *item, BadFlagsMarkItem bad_fl
 
 }
 
-
 static int32_t mark_all_in_stockpile(color_ostream &out, PersistentDataItem & stockpile, int32_t &premarked_item_count, int32_t &item_count,  map<int32_t, bool> &tracked_item_map, bool should_melt)
 {
     DEBUG(perf,out).print("%s running mark_all_in_stockpile\nshould_melt=%d\n", plugin_name, should_melt);
@@ -429,10 +406,8 @@ static int32_t mark_all_in_stockpile(color_ostream &out, PersistentDataItem & st
 
     int spid = get_config_val(stockpile, STOCKPILE_CONFIG_ID);
     auto found = df::building::find(spid);
-    if (!isStockpile(found)){
-
+    if (!isStockpile(found))
         return 0;
-    }
 
     df::building_stockpilest * pile_cast = virtual_cast<df::building_stockpilest>(found);
 
@@ -451,7 +426,6 @@ static int32_t mark_all_in_stockpile(color_ostream &out, PersistentDataItem & st
     return marked_count;
 }
 
-
 static int32_t scan_stockpiles(color_ostream &out, bool should_melt, map<int32_t, int32_t> &item_count_piles, map<int32_t, int32_t> &premarked_item_count_piles,
                                                                 map<int32_t, int32_t> &marked_item_count_piles, map<int32_t, bool> &tracked_item_map) {
     DEBUG(perf,out).print("running scan_stockpiles\n");
@@ -518,8 +492,6 @@ static int32_t scan_count_all(color_ostream &out, bool should_melt, int32_t &mar
 
     map<int32_t, bool> tracked_item_map_piles;
 
-    tracked_item_map_piles.clear();
-
     newly_marked_items_piles = scan_stockpiles(out, should_melt, item_count_piles, premarked_item_count_piles, marked_item_count_piles, tracked_item_map_piles);
     marked_item_count_global = scan_all_melt_designated(out, tracked_item_map_piles);
 
@@ -557,20 +529,18 @@ static int32_t do_cycle(color_ostream &out) {
 }
 
 static int getSelectedStockpile(color_ostream &out) {
-    df::building *selected_bldg = NULL;
-    selected_bldg = Gui::getSelectedBuilding(out, true);
-    if (selected_bldg->getType() != df::building_type::Stockpile) {
+    df::building *bld = Gui::getSelectedBuilding(out, true);
+    if (!isStockpile(bld)) {
         DEBUG(status,out).print("Selected building is not stockpile\n");
         return -1;
     }
 
-    return selected_bldg->id;
+    return bld->id;
 }
 
 static PersistentDataItem *getSelectedStockpileConfig(color_ostream &out) {
     int32_t bldg_id = getSelectedStockpile(out);
     if (bldg_id == -1) {
-        DEBUG(status,out).print("Selected bldg invalid\n");
         return NULL;
     }
 
@@ -579,11 +549,10 @@ static PersistentDataItem *getSelectedStockpileConfig(color_ostream &out) {
     if (watched_stockpiles.count(bldg_id)) {
         c = &(watched_stockpiles[bldg_id]);
         return c;
-    } else {
-        DEBUG(status,out).print("No existing config\n");
-        return NULL;
     }
 
+    DEBUG(status,out).print("No existing config\n");
+    return NULL;
 }
 
 static void push_stockpile_config(lua_State *L, int id, bool monitored) {
@@ -671,7 +640,7 @@ static void automelt_setStockpileConfig(color_ostream &out, int id, bool monitor
     DEBUG(status,out).print("entering automelt_setStockpileConfig for id=%d and monitored=%d\n", id, monitored);
     validate_stockpile_configs(out);
     auto bldg = df::building::find(id);
-    bool isInvalidStockpile = !bldg || !isStockpile(bldg);
+    bool isInvalidStockpile = !isStockpile(bldg);
     bool hasNoData = !monitored;
     if (isInvalidStockpile || hasNoData) {
         DEBUG(cycle,out).print("calling remove_stockpile_config with id=%d monitored=%d\n", id, monitored);
@@ -767,7 +736,6 @@ static int automelt_getSelectedStockpileConfig(lua_State *L){
     return 1;
 }
 
-//TODO
 static int automelt_getItemCountsAndStockpileConfigs(lua_State *L) {
     color_ostream *out = Lua::GetOutput(L);
     if (!out)

plugins/autoslab.cpp

@@ -90,6 +90,8 @@ DFhackCExport command_result plugin_enable(color_ostream &out, bool enable)
         is_enabled = enable;
         DEBUG(status, out).print("%s from the API; persisting\n", is_enabled ? "enabled" : "disabled");
         set_config_bool(CONFIG_IS_ENABLED, is_enabled);
+        if (enable)
+            do_cycle(out);
     }
     else
     {

plugins/cleanowned.cpp

@@ -74,8 +74,6 @@ command_result df_cleanowned (color_ostream &out, vector <string> & parameters)
         return CR_FAILURE;
     }
 
-    out.print("Found total %zd items.\n", world->items.all.size());
-
     for (std::size_t i=0; i < world->items.all.size(); i++)
     {
         df::item * item = world->items.all[i];

plugins/examples/persistent_per_save_example.cpp

@@ -6,6 +6,7 @@
 // savegame that had this plugin enabled is loaded.
 
 #include <string>
+#include <unordered_map>
 #include <vector>
 
 #include "df/world.h"
@@ -18,6 +19,7 @@
 #include "modules/World.h"
 
 using std::string;
+using std::unordered_map;
 using std::vector;
 
 using namespace DFHack;
@@ -38,25 +40,50 @@ namespace DFHack {
 }
 
 static const string CONFIG_KEY = string(plugin_name) + "/config";
+static const string ELEM_CONFIG_KEY_PREFIX = string(plugin_name) + "/elem/";
 static PersistentDataItem config;
+static unordered_map<int, PersistentDataItem> elems;
+
 enum ConfigValues {
     CONFIG_IS_ENABLED = 0,
     CONFIG_SOMETHING_ELSE = 1,
 };
-static int get_config_val(int index) {
-    if (!config.isValid())
+
+enum ElemConfigValues {
+    ELEM_CONFIG_ID = 0,
+    ELEM_CONFIG_SOMETHING_ELSE = 1,
+};
+
+static int get_config_val(PersistentDataItem &c, int index) {
+    if (!c.isValid())
         return -1;
-    return config.ival(index);
+    return c.ival(index);
+}
+static bool get_config_bool(PersistentDataItem &c, int index) {
+    return get_config_val(c, index) == 1;
 }
-static bool get_config_bool(int index) {
-    return get_config_val(index) == 1;
+static void set_config_val(PersistentDataItem &c, int index, int value) {
+    if (c.isValid())
+        c.ival(index) = value;
 }
-static void set_config_val(int index, int value) {
-    if (config.isValid())
-        config.ival(index) = value;
+static void set_config_bool(PersistentDataItem &c, int index, bool value) {
+    set_config_val(c, index, value ? 1 : 0);
 }
-static void set_config_bool(int index, bool value) {
-    set_config_val(index, value ? 1 : 0);
+
+static PersistentDataItem & ensure_elem_config(color_ostream &out, int id) {
+    if (elems.count(id))
+        return elems[id];
+    string keyname = ELEM_CONFIG_KEY_PREFIX + int_to_string(id);
+    DEBUG(config,out).print("creating new persistent key for elem id %d\n", id);
+    elems.emplace(id, World::GetPersistentData(keyname, NULL));
+    return elems[id];
+}
+static void remove_elem_config(color_ostream &out, int id) {
+    if (!elems.count(id))
+        return;
+    DEBUG(config,out).print("removing persistent key for elem id %d\n", id);
+    World::DeletePersistentData(elems[id]);
+    elems.erase(id);
 }
 
 static const int32_t CYCLE_TICKS = 1200; // one day
@@ -87,7 +114,9 @@ DFhackCExport command_result plugin_enable(color_ostream &out, bool enable) {
         is_enabled = enable;
         DEBUG(config,out).print("%s from the API; persisting\n",
                                 is_enabled ? "enabled" : "disabled");
-        set_config_bool(CONFIG_IS_ENABLED, is_enabled);
+        set_config_bool(config, CONFIG_IS_ENABLED, is_enabled);
+        if (enable)
+            do_cycle(out);
     } else {
         DEBUG(config,out).print("%s from the API, but already %s; no action\n",
                                 is_enabled ? "enabled" : "disabled",
@@ -109,16 +138,27 @@ DFhackCExport command_result plugin_load_data (color_ostream &out) {
     if (!config.isValid()) {
         DEBUG(config,out).print("no config found in this save; initializing\n");
         config = World::AddPersistentData(CONFIG_KEY);
-        set_config_bool(CONFIG_IS_ENABLED, is_enabled);
-        set_config_val(CONFIG_SOMETHING_ELSE, 6000);
+        set_config_bool(config, CONFIG_IS_ENABLED, is_enabled);
+        set_config_val(config, CONFIG_SOMETHING_ELSE, 6000);
     }
 
     // we have to copy our enabled flag into the global plugin variable, but
     // all the other state we can directly read/modify from the persistent
     // data structure.
-    is_enabled = get_config_bool(CONFIG_IS_ENABLED);
+    is_enabled = get_config_bool(config, CONFIG_IS_ENABLED);
     DEBUG(config,out).print("loading persisted enabled state: %s\n",
                             is_enabled ? "true" : "false");
+
+    // load other config elements, if applicable
+    elems.clear();
+    vector<PersistentDataItem> elem_configs;
+    World::GetPersistentData(&elem_configs, ELEM_CONFIG_KEY_PREFIX, true);
+    const size_t num_elem_configs = elem_configs.size();
+    for (size_t idx = 0; idx < num_elem_configs; ++idx) {
+        auto &c = elem_configs[idx];
+        elems.emplace(get_config_val(c, ELEM_CONFIG_ID), c);
+    }
+
     return CR_OK;
 }
 
@@ -150,8 +190,8 @@ static command_result do_command(color_ostream &out, vector<string> &parameters)
 
     // TODO: configuration logic
     // simple commandline parsing can be done in C++, but there are lua libraries
-    // that can easily handle more complex commandlines. see the blueprint plugin
-    // for an example.
+    // that can easily handle more complex commandlines. see the seedwatch plugin
+    // for a simple example.
 
     return CR_OK;
 }

plugins/lua/autolabor.lua

@@ -4,16 +4,11 @@ local gui = require('gui')
 local overlay = require('plugins.overlay')
 local widgets = require('gui.widgets')
 
-local function is_labor_panel_visible()
-    local info = df.global.game.main_interface.info
-    return info.open and info.current_mode == df.info_interface_mode_type.LABOR
-end
-
 AutolaborOverlay = defclass(AutolaborOverlay, overlay.OverlayWidget)
 AutolaborOverlay.ATTRS{
     default_pos={x=7,y=-13},
     default_enabled=true,
-    viewscreens='dwarfmode',
+    viewscreens='dwarfmode/Info/LABOR',
     frame={w=29, h=5},
     frame_style=gui.MEDIUM_FRAME,
     frame_background=gui.CLEAR_PEN,
@@ -34,9 +29,7 @@ function AutolaborOverlay:init()
 end
 
 function AutolaborOverlay:render(dc)
-    if not is_labor_panel_visible() or not isEnabled() then
-        return false
-    end
+    if not isEnabled() then return false end
     AutolaborOverlay.super.render(self, dc)
 end
 

plugins/lua/automelt.lua

@@ -17,8 +17,12 @@ end
 local function do_set_stockpile_config(var_name, val, stockpiles)
     for _,bspec in ipairs(argparse.stringList(stockpiles)) do
         local config = automelt_getStockpileConfig(bspec)
-        config[var_name] = val
-        automelt_setStockpileConfig(config.id, config.monitor, config.melt)
+        if not config then
+            dfhack.printerr('invalid stockpile: '..tostring(bspec))
+        else
+            config[var_name] = val
+            automelt_setStockpileConfig(config.id, config.monitor, config.melt)
+        end
     end
 end
 

plugins/lua/orders.lua

@@ -9,11 +9,6 @@ local widgets = require('gui.widgets')
 -- OrdersOverlay
 --
 
-local function is_orders_panel_visible()
-    local info = df.global.game.main_interface.info
-    return info.open and info.current_mode == df.info_interface_mode_type.WORK_ORDERS
-end
-
 local function do_sort()
     dfhack.run_command('orders', 'sort')
 end
@@ -49,7 +44,7 @@ OrdersOverlay = defclass(OrdersOverlay, overlay.OverlayWidget)
 OrdersOverlay.ATTRS{
     default_pos={x=53,y=-6},
     default_enabled=true,
-    viewscreens='dwarfmode',
+    viewscreens='dwarfmode/Info/WORK_ORDERS',
     frame={w=30, h=4},
     frame_style=gui.MEDIUM_FRAME,
     frame_background=gui.CLEAR_PEN,
@@ -84,16 +79,6 @@ function OrdersOverlay:init()
     }
 end
 
-function OrdersOverlay:render(dc)
-    if not is_orders_panel_visible() then return false end
-    OrdersOverlay.super.render(self, dc)
-end
-
-function OrdersOverlay:onInput(keys)
-    if not is_orders_panel_visible() then return false end
-    OrdersOverlay.super.onInput(self, keys)
-end
-
 OVERLAY_WIDGETS = {
     overlay=OrdersOverlay,
 }

plugins/lua/overlay.lua

@@ -81,19 +81,18 @@ function normalize_list(element_or_list)
     return {element_or_list}
 end
 
--- normalize "short form" viewscreen names to "long form"
+-- normalize "short form" viewscreen names to "long form" and remove any focus
 local function normalize_viewscreen_name(vs_name)
-    if vs_name == 'all' or vs_name:match('viewscreen_.*st') then
-        return vs_name
+    if vs_name == 'all' or vs_name:match('^viewscreen_.*st') then
+        return vs_name:match('^[^/]+')
     end
-    return 'viewscreen_' .. vs_name .. 'st'
+    return 'viewscreen_' .. vs_name:match('^[^/]+') .. 'st'
 end
 
--- reduce "long form" viewscreen names to "short form"
+-- reduce "long form" viewscreen names to "short form"; keep focus
 function simplify_viewscreen_name(vs_name)
-    _,_,short_name = vs_name:find('^viewscreen_(.*)st$')
-    if short_name then return short_name end
-    return vs_name
+    local short_name = vs_name:match('^viewscreen_([^/]+)st')
+    return short_name or vs_name
 end
 
 local function is_empty(tbl)
@@ -241,10 +240,23 @@ local function do_list(args)
     end
 end
 
+local function get_focus_strings(viewscreens)
+    local focus_strings = nil
+    for _,vs in ipairs(viewscreens) do
+        if vs:match('/') then
+            focus_strings = focus_strings or {}
+            vs = simplify_viewscreen_name(vs)
+            table.insert(focus_strings, vs)
+        end
+    end
+    return focus_strings
+end
+
 local function load_widget(name, widget_class)
     local widget = widget_class{name=name}
     widget_db[name] = {
         widget=widget,
+        focus_strings=get_focus_strings(normalize_list(widget.viewscreens)),
         next_update_ms=widget.overlay_onupdate and 0 or math.huge,
     }
     if not overlay_config[name] then overlay_config[name] = {} end
@@ -426,12 +438,30 @@ function update_hotspot_widgets()
     end
 end
 
+local function matches_focus_strings(db_entry, vs_name)
+    if not db_entry.focus_strings then return true end
+    local matched = true
+    local simple_vs_name = simplify_viewscreen_name(vs_name)
+    for _,fs in ipairs(db_entry.focus_strings) do
+        if fs:startswith(simple_vs_name) then
+            matched = false
+            if dfhack.gui.matchFocusString(fs, vs) then
+                return true
+            end
+        end
+    end
+    return matched
+end
+
 local function _update_viewscreen_widgets(vs_name, vs, now_ms)
     local vs_widgets = active_viewscreen_widgets[vs_name]
     if not vs_widgets then return end
     now_ms = now_ms or dfhack.getTickCount()
     for name,db_entry in pairs(vs_widgets) do
-        if do_update(name, db_entry, now_ms, vs) then return end
+        if matches_focus_strings(db_entry, vs_name) and
+                do_update(name, db_entry, now_ms, vs) then
+            return
+        end
     end
     return now_ms
 end
@@ -439,7 +469,9 @@ end
 function update_viewscreen_widgets(vs_name, vs)
     if triggered_screen_has_lock() then return end
     local now_ms = _update_viewscreen_widgets(vs_name, vs, nil)
-    _update_viewscreen_widgets('all', vs, now_ms)
+    if now_ms then
+        _update_viewscreen_widgets('all', vs, now_ms)
+    end
 end
 
 local function _feed_viewscreen_widgets(vs_name, keys)
@@ -447,7 +479,8 @@ local function _feed_viewscreen_widgets(vs_name, keys)
     if not vs_widgets then return false end
     for _,db_entry in pairs(vs_widgets) do
         local w = db_entry.widget
-        if detect_frame_change(w, function() return w:onInput(keys) end) then
+        if matches_focus_strings(db_entry, vs_name) and
+                detect_frame_change(w, function() return w:onInput(keys) end) then
             return true
         end
     end
@@ -465,7 +498,9 @@ local function _render_viewscreen_widgets(vs_name, dc)
     dc = dc or gui.Painter.new()
     for _,db_entry in pairs(vs_widgets) do
         local w = db_entry.widget
-        detect_frame_change(w, function() w:render(dc) end)
+        if matches_focus_strings(db_entry, vs_name) then
+            detect_frame_change(w, function() w:render(dc) end)
+        end
     end
 end
 

plugins/nestboxes.cpp

@@ -70,6 +70,8 @@ DFhackCExport command_result plugin_enable(color_ostream &out, bool enable) {
         DEBUG(config,out).print("%s from the API; persisting\n",
                                 is_enabled ? "enabled" : "disabled");
         set_config_bool(config, CONFIG_IS_ENABLED, is_enabled);
+        if (enable)
+            do_cycle(out);
     } else {
         DEBUG(config,out).print("%s from the API, but already %s; no action\n",
                                 is_enabled ? "enabled" : "disabled",

plugins/seedwatch.cpp

@@ -99,7 +99,7 @@ static const int32_t CYCLE_TICKS = 1200;
 static int32_t cycle_timestamp = 0;  // world->frame_counter at last cycle
 
 static command_result do_command(color_ostream &out, vector<string> &parameters);
-static void do_cycle(color_ostream &out, int32_t *num_enabled_seeds, int32_t *num_disabled_seeds);
+static void do_cycle(color_ostream &out, int32_t *num_enabled_seeds = NULL, int32_t *num_disabled_seeds = NULL);
 static void seedwatch_setTarget(color_ostream &out, string name, int32_t num);
 
 DFhackCExport command_result plugin_init(color_ostream &out, std::vector <PluginCommand> &commands) {
@@ -149,7 +149,7 @@ DFhackCExport command_result plugin_enable(color_ostream &out, bool enable) {
                                 is_enabled ? "enabled" : "disabled");
         set_config_bool(config, CONFIG_IS_ENABLED, is_enabled);
         if (enable)
-            seedwatch_setTarget(out, "all", DEFAULT_TARGET);
+            do_cycle(out);
     } else {
         DEBUG(config,out).print("%s from the API, but already %s; no action\n",
                                 is_enabled ? "enabled" : "disabled",
@@ -174,26 +174,27 @@ DFhackCExport command_result plugin_load_data (color_ostream &out) {
             world_plant_ids[plant->id] = i;
     }
 
+    watched_seeds.clear();
+    vector<PersistentDataItem> seed_configs;
+    World::GetPersistentData(&seed_configs, SEED_CONFIG_KEY_PREFIX, true);
+    const size_t num_seed_configs = seed_configs.size();
+    for (size_t idx = 0; idx < num_seed_configs; ++idx) {
+        auto &c = seed_configs[idx];
+        watched_seeds.emplace(get_config_val(c, SEED_CONFIG_ID), c);
+    }
+
     config = World::GetPersistentData(CONFIG_KEY);
 
     if (!config.isValid()) {
         DEBUG(config,out).print("no config found in this save; initializing\n");
         config = World::AddPersistentData(CONFIG_KEY);
         set_config_bool(config, CONFIG_IS_ENABLED, is_enabled);
+        seedwatch_setTarget(out, "all", DEFAULT_TARGET);
     }
 
     is_enabled = get_config_bool(config, CONFIG_IS_ENABLED);
     DEBUG(config,out).print("loading persisted enabled state: %s\n",
                             is_enabled ? "true" : "false");
-    watched_seeds.clear();
-    vector<PersistentDataItem> seed_configs;
-    World::GetPersistentData(&seed_configs, SEED_CONFIG_KEY_PREFIX, true);
-    const size_t num_seed_configs = seed_configs.size();
-    for (size_t idx = 0; idx < num_seed_configs; ++idx) {
-        auto &c = seed_configs[idx];
-        watched_seeds.emplace(get_config_val(c, SEED_CONFIG_ID), c);
-    }
-
     return CR_OK;
 }