From 0058331aeb027f7441a04d99cc7c1e445bd896d9 Mon Sep 17 00:00:00 2001
From: Simon Tatham <anakin@pobox.com>
Date: Fri, 21 Apr 2023 15:50:05 +0100
Subject: New backend functions: get_prefs and set_prefs.

These are similar to the existing pair configure() and custom_params()
in that get_prefs() returns an array of config_item describing a set
of dialog-box controls to present to the user, and set_prefs()
receives the same array with answers filled in and implements the
answers. But where configure() and custom_params() operate on a
game_params structure, the new pair operate on a game_ui, and are
intended to permit GUI configuration of all the settings I just moved
into that structure.

However, nothing actually _calls_ these routines yet. All I've done in
this commit is to add them to 'struct game' and implement them for the
functions that need them.

Also, config_item has new fields, permitting each config option to
define a machine-readable identifying keyword as well as the
user-facing description. For options of type C_CHOICES, each choice
also has a keyword. These keyword fields are only defined at all by
the new get_prefs() function - they're left uninitialised in existing
uses of the dialog system. The idea is to use them when writing out
the user's preferences into a configuration file on disk, although I
haven't actually done any of that work in this commit.
---
 unfinished/group.c    | 1 +
 unfinished/separate.c | 1 +
 unfinished/slide.c    | 1 +
 unfinished/sokoban.c  | 1 +
 4 files changed, 4 insertions(+)

(limited to 'unfinished')

diff --git a/unfinished/group.c b/unfinished/group.c
index dfa466d..7faee89 100644
--- a/unfinished/group.c
+++ b/unfinished/group.c
@@ -2323,6 +2323,7 @@ const struct game thegame = {
     free_game,
     true, solve_game,
     true, game_can_format_as_text_now, game_text_format,
+    NULL, NULL, /* get_prefs, set_prefs */
     new_ui,
     free_ui,
     NULL, /* encode_ui */
diff --git a/unfinished/separate.c b/unfinished/separate.c
index 635d7c6..6ca0725 100644
--- a/unfinished/separate.c
+++ b/unfinished/separate.c
@@ -835,6 +835,7 @@ const struct game thegame = {
     free_game,
     false, solve_game,
     false, game_can_format_as_text_now, game_text_format,
+    NULL, NULL, /* get_prefs, set_prefs */
     new_ui,
     free_ui,
     NULL, /* encode_ui */
diff --git a/unfinished/slide.c b/unfinished/slide.c
index 6368eba..c797d7e 100644
--- a/unfinished/slide.c
+++ b/unfinished/slide.c
@@ -2328,6 +2328,7 @@ const struct game thegame = {
     free_game,
     true, solve_game,
     true, game_can_format_as_text_now, game_text_format,
+    NULL, NULL, /* get_prefs, set_prefs */
     new_ui,
     free_ui,
     NULL, /* encode_ui */
diff --git a/unfinished/sokoban.c b/unfinished/sokoban.c
index 3b847af..1f3c688 100644
--- a/unfinished/sokoban.c
+++ b/unfinished/sokoban.c
@@ -1450,6 +1450,7 @@ const struct game thegame = {
     free_game,
     false, solve_game,
     false, game_can_format_as_text_now, game_text_format,
+    NULL, NULL, /* get_prefs, set_prefs */
     new_ui,
     free_ui,
     NULL, /* encode_ui */
-- 
cgit v1.1