From 7ff8ef94c05ae6386463b63f3ba25add52340d8b Mon Sep 17 00:00:00 2001 From: Vito Caputo Date: Sat, 12 Mar 2022 17:29:26 -0800 Subject: til_settings: always describe relevant settings The existing iterative *_setup() interface only described settings not found, quietly accepting usable settings already present in the til_settings_t. This worked fine for the existing interactive text setup thing, but it's especially problematic for providing a GUI setup frontend. This commit makes it so the *_setup() methods always describe undescribed settings they recognize, leaving the setup frontend loop calling into the *_setup() methods to both apply the description validation if wanted and actually tie the description to respective setting returned by the _setup() methods as being related to the returned description. A new helper called til_settings_get_and_describe_value() has been introduced primarily for use of module setup methods to simplify this nonsense, replacing the til_settings_get_value() calls and surrounding logic, but retaining the til_setting_desc_t definitions largely verbatim. This also results in discarding of some ad-hoc til_setting_desc_check() calls, now that there's a centralized place where settings become "described" (setup_interactively in the case of rototiller). Now a GUI frontend (like glimmer) would just provide its own setup_interactively() equivalent for constructing its widgets for a given *_setup() method's chain of returned descs. Whereas in the past this wasn't really feasible unless there was never going to be pre-supplied settings. I suspect the til_setting_desc_check() integration into setup_interactively() needs more work, but I think this is good enough for now and I'm out of spare time for the moment. --- src/modules/sparkler/sparkler.c | 127 ++++++++++++++++++---------------------- 1 file changed, 58 insertions(+), 69 deletions(-) (limited to 'src/modules/sparkler') diff --git a/src/modules/sparkler/sparkler.c b/src/modules/sparkler/sparkler.c index 8b8681f..6a4b604 100644 --- a/src/modules/sparkler/sparkler.c +++ b/src/modules/sparkler/sparkler.c @@ -96,83 +96,75 @@ static void sparkler_render_fragment(void *context, unsigned ticks, unsigned cpu /* Settings hooks for configurable variables */ -static int sparkler_setup(const til_settings_t *settings, til_setting_desc_t **next_setting) +static int sparkler_setup(const til_settings_t *settings, const til_setting_t **res_setting, const til_setting_desc_t **res_desc) { const char *show_bsp_leafs; const char *show_bsp_matches; const char *values[] = { - "off", - "on", - NULL + "off", + "on", + NULL }; + int r; /* TODO: return -EINVAL on parse errors? */ - show_bsp_leafs = til_settings_get_value(settings, "show_bsp_leafs"); - if (!show_bsp_leafs) { - int r; - - r = til_setting_desc_clone(&(til_setting_desc_t){ - .name = "Show BSP Leaf Node Bounding Boxes", - .key = "show_bsp_leafs", - .preferred = "off", - .values = values, - }, next_setting); - if (r < 0) - return r; - - return 1; - } + r = til_settings_get_and_describe_value(settings, + &(til_setting_desc_t){ + .name = "Show BSP Leaf Node Bounding Boxes", + .key = "show_bsp_leafs", + .preferred = "off", + .values = values + }, + &show_bsp_leafs, + res_setting, + res_desc); + if (r) + return r; if (!strcasecmp(show_bsp_leafs, "on")) { + const char *depth_values[] = { + "0", + "4", + "6", + "8", + "10", + NULL + }; const char *show_bsp_leafs_min_depth; sparkler_conf.show_bsp_leafs = 1; - show_bsp_leafs_min_depth = til_settings_get_value(settings, "show_bsp_leafs_min_depth"); - if (!show_bsp_leafs_min_depth) { - const char *depth_values[] = { - "0", - "4", - "6", - "8", - "10", - NULL - }; - int r; - - r = til_setting_desc_clone(&(til_setting_desc_t){ + r = til_settings_get_and_describe_value(settings, + &(til_setting_desc_t){ .name = "Show BSP Leaf Node Bounding Boxes Minimum Depth", .key = "show_bsp_leafs_min_depth", .preferred = "8", - .values = depth_values, - }, next_setting); - if (r < 0) - return r; - - return 1; - } + .values = depth_values + }, + &show_bsp_leafs_min_depth, + res_setting, + res_desc); + if (r) + return r; sscanf(show_bsp_leafs_min_depth, "%u", &sparkler_conf.show_bsp_leafs_min_depth); } else { sparkler_conf.show_bsp_leafs = 0; } - show_bsp_matches = til_settings_get_value(settings, "show_bsp_matches"); - if (!show_bsp_matches) { - int r; - - r = til_setting_desc_clone(&(til_setting_desc_t){ - .name = "Show BSP Search Matches", - .key = "show_bsp_matches", - .preferred = "off", - .values = values, - }, next_setting); - if (r < 0) - return r; - - return 1; - } + r = til_settings_get_and_describe_value(settings, + &(til_setting_desc_t){ + .name = "Show BSP Search Matches", + .key = "show_bsp_matches", + .preferred = "off", + .values = values + }, + &show_bsp_matches, + res_setting, + res_desc); + if (r) + return r; if (!strcasecmp(show_bsp_matches, "on")) sparkler_conf.show_bsp_matches = 1; @@ -182,21 +174,18 @@ static int sparkler_setup(const til_settings_t *settings, til_setting_desc_t **n if (!strcasecmp(show_bsp_matches, "on")) { const char *show_bsp_matches_affected_only; - show_bsp_matches_affected_only = til_settings_get_value(settings, "show_bsp_matches_affected_only"); - if (!show_bsp_matches_affected_only) { - int r; - - r = til_setting_desc_clone(&(til_setting_desc_t){ - .name = "Show Only Affected BSP Search Matches", - .key = "show_bsp_matches_affected_only", - .preferred = "off", - .values = values, - }, next_setting); - if (r < 0) - return r; - - return 1; - } + r = til_settings_get_and_describe_value(settings, + &(til_setting_desc_t){ + .name = "Show Only Affected BSP Search Matches", + .key = "show_bsp_matches_affected_only", + .preferred = "off", + .values = values + }, + &show_bsp_matches_affected_only, + res_setting, + res_desc); + if (r) + return r; if (!strcasecmp(show_bsp_matches_affected_only, "on")) sparkler_conf.show_bsp_matches_affected_only = 1; -- cgit v1.2.3