From ac99a38175c78612d058b40e7e1219d440b3d7ce Mon Sep 17 00:00:00 2001 From: rubidium42 Date: Sat, 22 May 2021 08:32:51 +0200 Subject: Cleanup: remove and/or fix some confusing comments The comments for SettingDescType; it is a byte, so not 4 bytes and since it is not a flag there are about 250 other possibilities left instead of 9. SettingGuiFlag is uint16 so has 2 bytes allocated. SettingDescGlobVarList and related comments imply that global vars cannot be used elsewhere, but they are used for settings just fine. Even then the type is not used anywhere else but the definition of the table. --- src/table/misc_settings.ini | 2 +- src/table/settings.h.preamble | 4 +--- src/table/win32_settings.ini | 2 +- 3 files changed, 3 insertions(+), 5 deletions(-) (limited to 'src/table') diff --git a/src/table/misc_settings.ini b/src/table/misc_settings.ini index 37994644a..2613b4ddd 100644 --- a/src/table/misc_settings.ini +++ b/src/table/misc_settings.ini @@ -16,7 +16,7 @@ extern bool _allow_hidpi_window; #define WITHOUT_COCOA #endif -static const SettingDescGlobVarList _misc_settings[] = { +static const SettingDesc _misc_settings[] = { [post-amble] }; [templates] diff --git a/src/table/settings.h.preamble b/src/table/settings.h.preamble index 826b272d7..6410b0360 100644 --- a/src/table/settings.h.preamble +++ b/src/table/settings.h.preamble @@ -23,9 +23,7 @@ static size_t ConvertLandscape(const char *value); * The macros can be grouped depending on where the config variable is * stored: * 1. SDTG_something - * These are for global variables, so this is the one you will use - * for a #SettingDescGlobVarList section. Here 'var' refers to a - * global variable. + * These are for global variables. Here 'var' refers to a global variable. * 2. SDTC_something * These are for client-only variables. Here the 'var' refers to an * entry inside _settings_client. diff --git a/src/table/win32_settings.ini b/src/table/win32_settings.ini index c3cc45014..963a11df8 100644 --- a/src/table/win32_settings.ini +++ b/src/table/win32_settings.ini @@ -9,7 +9,7 @@ #if defined(_WIN32) && !defined(DEDICATED) extern bool _window_maximize; -static const SettingDescGlobVarList _win32_settings[] = { +static const SettingDesc _win32_settings[] = { [post-amble] }; #endif /* _WIN32 */ -- cgit v1.2.3-70-g09d2