summaryrefslogtreecommitdiff
path: root/src/wx
diff options
context:
space:
mode:
authorCarl Hetherington <cth@carlh.net>2025-03-03 20:51:48 +0100
committerCarl Hetherington <cth@carlh.net>2025-03-08 00:12:07 +0100
commit3f2675aab119e55f958563e2fe6949192a2b976d (patch)
tree2ed8c7e2a0eb7147ab9b798e934d1f7a4ce453a9 /src/wx
parentf180d199dcb0217ea9750ab0a788217689733c41 (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.h18
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);
}