diff options
| author | Carl Hetherington <cth@carlh.net> | 2025-03-03 20:51:48 +0100 |
|---|---|---|
| committer | Carl Hetherington <cth@carlh.net> | 2025-03-08 00:12:07 +0100 |
| commit | 3f2675aab119e55f958563e2fe6949192a2b976d (patch) | |
| tree | 2ed8c7e2a0eb7147ab9b798e934d1f7a4ce453a9 /src/wx | |
| parent | f180d199dcb0217ea9750ab0a788217689733c41 (diff) | |
Don't use an optional for the Grok config object.
I'm not sure why I did it like this in the first place - perhaps so that
if the API endpoint changes there aren't so many old values stuck in
config files all over the place? Anyway, it seems cleaner to do it like
this, as it's how we handle the other config.
Diffstat (limited to 'src/wx')
| -rw-r--r-- | src/wx/grok/gpu_config_panel.h | 18 |
1 files changed, 9 insertions, 9 deletions
diff --git a/src/wx/grok/gpu_config_panel.h b/src/wx/grok/gpu_config_panel.h index df38f9373..3f4d0312c 100644 --- a/src/wx/grok/gpu_config_panel.h +++ b/src/wx/grok/gpu_config_panel.h @@ -63,7 +63,7 @@ public: void update() { - auto grok = Config::instance()->grok().get_value_or({}); + auto grok = Config::instance()->grok(); auto lister_binary = grok.binary_location / "gpu_lister"; auto lister_file = grok.binary_location / "gpus.txt"; if (boost::filesystem::exists(lister_binary)) { @@ -88,7 +88,7 @@ private: { auto selection = _combo_box->GetSelection(); if (selection != wxNOT_FOUND) { - auto grok = Config::instance()->grok().get_value_or({}); + auto grok = Config::instance()->grok(); grok.selected = selection; Config::instance()->set_grok(grok); } @@ -155,7 +155,7 @@ private: void setup_sensitivity() { - auto grok = Config::instance()->grok().get_value_or({}); + auto grok = Config::instance()->grok(); _binary_location->Enable(grok.enable); _gpu_list_control->Enable(grok.enable); @@ -165,7 +165,7 @@ private: void config_changed() override { - auto grok = Config::instance()->grok().get_value_or({}); + auto grok = Config::instance()->grok(); checked_set(_enable_gpu, grok.enable); _binary_location->SetPath(std_to_wx(grok.binary_location.string())); @@ -177,7 +177,7 @@ private: void enable_gpu_changed() { - auto grok = Config::instance()->grok().get_value_or({}); + auto grok = Config::instance()->grok(); grok.enable = _enable_gpu->GetValue(); Config::instance()->set_grok(grok); @@ -186,7 +186,7 @@ private: void binary_location_changed() { - auto grok = Config::instance()->grok().get_value_or({}); + auto grok = Config::instance()->grok(); grok.binary_location = wx_to_std(_binary_location->GetPath()); Config::instance()->set_grok(grok); @@ -195,20 +195,20 @@ private: void server_changed() { - auto grok = Config::instance()->grok().get_value_or({}); + auto grok = Config::instance()->grok(); grok.licence_server = wx_to_std(_server->GetValue()); Config::instance()->set_grok(grok); } void port_changed() { - auto grok = Config::instance()->grok().get_value_or({}); + auto grok = Config::instance()->grok(); Config::instance()->set_grok(grok); } void licence_changed() { - auto grok = Config::instance()->grok().get_value_or({}); + auto grok = Config::instance()->grok(); grok.licence = _licence->get(); Config::instance()->set_grok(grok); } |
