Fix #9386: compilers failing to compile with LTO by using variants instead of new + unique_ptr

With std::variant all memory can be figured out at compile time, so the compiler needs to keep track of fewer elements. It also saves out a unique_ptr and its memory management, over a slight impact for resolving a setting.

7 files changed, 7 insertions, 7 deletions

diff --git a/src/table/settings/company_settings.ini b/src/table/settings/company_settings.ini
index 71f99d5a2..e5019305f 100644
--- a/src/table/settings/company_settings.ini
+++ b/src/table/settings/company_settings.ini
@@ -12,7 +12,7 @@ static void UpdateAllServiceInterval(int32 new_value);
 static bool CanUpdateServiceInterval(VehicleType type, int32 &new_value);
 static void UpdateServiceInterval(VehicleType type, int32 new_value);
 
-static const SettingTable _company_settings{
+static const SettingVariant _company_settings[] = {
 [post-amble]
 };
 [templates]
diff --git a/src/table/settings/currency_settings.ini b/src/table/settings/currency_settings.ini
index c8ba3d76c..d0950db8e 100644
--- a/src/table/settings/currency_settings.ini
+++ b/src/table/settings/currency_settings.ini
@@ -7,7 +7,7 @@
 ; Settings for the in-game custom currency.
 
 [pre-amble]
-static const SettingTable _currency_settings{
+static const SettingVariant _currency_settings[] = {
 [post-amble]
 };
 [templates]
diff --git a/src/table/settings/gameopt_settings.ini b/src/table/settings/gameopt_settings.ini
index ce28def63..447400623 100644
--- a/src/table/settings/gameopt_settings.ini
+++ b/src/table/settings/gameopt_settings.ini
@@ -33,7 +33,7 @@ static std::initializer_list<const char*> _osk_activation{"disabled", "double",
 static std::initializer_list<const char*> _settings_profiles{"easy", "medium", "hard"};
 static std::initializer_list<const char*> _news_display{ "off", "summarized", "full"};
 
-static const SettingTable _gameopt_settings{
+static const SettingVariant _gameopt_settings[] = {
 /* In version 4 a new difficulty setting has been added to the difficulty settings,
  * town attitude towards demolishing. Needs special handling because some dimwit thought
  * it funny to have the GameDifficulty struct be an array while it is a struct of
diff --git a/src/table/settings/misc_settings.ini b/src/table/settings/misc_settings.ini
index 453c6e816..9f73573b2 100644
--- a/src/table/settings/misc_settings.ini
+++ b/src/table/settings/misc_settings.ini
@@ -20,7 +20,7 @@ extern bool _allow_hidpi_window;
 #define WITHOUT_COCOA
 #endif
 
-static const SettingTable _misc_settings{
+static const SettingVariant _misc_settings[] = {
 [post-amble]
 };
 [templates]
diff --git a/src/table/settings/settings.ini b/src/table/settings/settings.ini
index 6fe4d9d51..a0a835364 100644
--- a/src/table/settings/settings.ini
+++ b/src/table/settings/settings.ini
@@ -49,7 +49,7 @@ static void UpdateClientConfigValues();
  * assigns its own value. If the setting was company-based, that would mean that
  * vehicles could decide on different moments that they are heading back to a
  * service depot, causing desyncs on a massive scale. */
-const SettingTable _settings{
+static const SettingVariant _settings[] = {
 [post-amble]
 };
 [templates]
diff --git a/src/table/settings/win32_settings.ini b/src/table/settings/win32_settings.ini
index df1e0cb27..51b579629 100644
--- a/src/table/settings/win32_settings.ini
+++ b/src/table/settings/win32_settings.ini
@@ -12,7 +12,7 @@
 #if defined(_WIN32) && !defined(DEDICATED)
 extern bool _window_maximize;
 
-static const SettingTable _win32_settings{
+static const SettingVariant _win32_settings[] = {
 [post-amble]
 };
 #endif /* _WIN32 */
diff --git a/src/table/settings/window_settings.ini b/src/table/settings/window_settings.ini
index 3a7669754..d9327aa92 100644
--- a/src/table/settings/window_settings.ini
+++ b/src/table/settings/window_settings.ini
@@ -9,7 +9,7 @@
 
 [pre-amble]
 
-static const SettingTable _window_settings{
+static const SettingVariant _window_settings[] = {
 [post-amble]
 };
 [templates]