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/lib/grok | |
| 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/lib/grok')
| -rw-r--r-- | src/lib/grok/context.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/lib/grok/context.h b/src/lib/grok/context.h index 602c8b13f..b31867cf6 100644 --- a/src/lib/grok/context.h +++ b/src/lib/grok/context.h @@ -99,7 +99,7 @@ public: explicit GrokContext(DcpomaticContext* dcpomatic_context) : _dcpomatic_context(dcpomatic_context) { - auto grok = Config::instance()->grok().get_value_or({}); + auto grok = Config::instance()->grok(); if (!grok.enable) { return; } @@ -216,7 +216,7 @@ public: auto s = dcpv.get_size(); _dcpomatic_context->set_dimensions(s.width, s.height); - auto grok = Config::instance()->grok().get_value_or({}); + auto grok = Config::instance()->grok(); if (!_messenger->launch_grok( _dcpomatic_context->location, _dcpomatic_context->width, |