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.

1 files changed, 2 insertions, 2 deletions

diff --git a/src/lib/config.h b/src/lib/config.h
index d9a95ebfd..b2a979ffa 100644
--- a/src/lib/config.h
+++ b/src/lib/config.h
@@ -666,7 +666,7 @@ public:
 		std::string licence;
 	};
 
-	boost::optional<Grok> grok() const {
+	Grok grok() const {
 		return _grok;
 	}
 #endif
@@ -1495,7 +1495,7 @@ private:
 	bool _layout_for_short_screen;
 
 #ifdef DCPOMATIC_GROK
-	boost::optional<Grok> _grok;
+	Grok _grok;
 #endif
 
 	ExportConfig _export;