summaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
Diffstat (limited to 'src')
-rw-r--r--src/lib/config.cc6
-rw-r--r--src/lib/config.h4
-rw-r--r--src/lib/grok/context.h4
-rw-r--r--src/lib/grok_j2k_encoder_thread.cc2
-rw-r--r--src/lib/j2k_encoder.cc10
-rw-r--r--src/lib/util.cc4
-rw-r--r--src/wx/grok/gpu_config_panel.h18
7 files changed, 23 insertions, 25 deletions
diff --git a/src/lib/config.cc b/src/lib/config.cc
index 3227ea433..49c64e5b6 100644
--- a/src/lib/config.cc
+++ b/src/lib/config.cc
@@ -238,7 +238,7 @@ Config::set_defaults()
set_cover_sheet_to_default();
#ifdef DCPOMATIC_GROK
- _grok = boost::none;
+ _grok = {};
#endif
_main_divider_sash_position = {};
@@ -1151,9 +1151,7 @@ Config::write_config() const
cxml::add_text_child(root, "LayoutForShortScreen", _layout_for_short_screen ? "1" : "0");
#ifdef DCPOMATIC_GROK
- if (_grok) {
- _grok->as_xml(cxml::add_child(root, "Grok"));
- }
+ _grok.as_xml(cxml::add_child(root, "Grok"));
#endif
_export.write(cxml::add_child(root, "Export"));
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;
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,
diff --git a/src/lib/grok_j2k_encoder_thread.cc b/src/lib/grok_j2k_encoder_thread.cc
index e6c256f11..d6825113c 100644
--- a/src/lib/grok_j2k_encoder_thread.cc
+++ b/src/lib/grok_j2k_encoder_thread.cc
@@ -62,7 +62,7 @@ try
LOG_TIMING("encoder-pop thread=%1 frame=%2 eyes=%3", thread_id(), frame.index(), static_cast<int>(frame.eyes()));
- auto grok = Config::instance()->grok().get_value_or({});
+ auto grok = Config::instance()->grok();
if (_context->launch(frame, grok.selected) && _context->scheduleCompress(frame)) {
frame_guard.cancel();
diff --git a/src/lib/j2k_encoder.cc b/src/lib/j2k_encoder.cc
index 309fce0b3..50452fbad 100644
--- a/src/lib/j2k_encoder.cc
+++ b/src/lib/j2k_encoder.cc
@@ -98,7 +98,7 @@ J2KEncoder::J2KEncoder(shared_ptr<const Film> film, Writer& writer)
#endif
{
#ifdef DCPOMATIC_GROK
- auto grok = Config::instance()->grok().get_value_or({});
+ auto grok = Config::instance()->grok();
_dcpomatic_context = new grk_plugin::DcpomaticContext(film, writer, _history, grok.binary_location);
if (grok.enable) {
_context = new grk_plugin::GrokContext(_dcpomatic_context);
@@ -135,7 +135,7 @@ J2KEncoder::servers_list_changed()
{
auto config = Config::instance();
#ifdef DCPOMATIC_GROK
- auto const grok_enable = config->grok().get_value_or({}).enable;
+ auto const grok_enable = config->grok().enable;
#else
auto const grok_enable = false;
#endif
@@ -162,7 +162,7 @@ void
J2KEncoder::pause()
{
#ifdef DCPOMATIC_GROK
- if (!Config::instance()->grok().get_value_or({}).enable) {
+ if (!Config::instance()->grok().enable) {
return;
}
return;
@@ -183,7 +183,7 @@ J2KEncoder::pause()
void J2KEncoder::resume()
{
#ifdef DCPOMATIC_GROK
- if (!Config::instance()->grok().get_value_or({}).enable) {
+ if (!Config::instance()->grok().enable) {
return;
}
@@ -226,7 +226,7 @@ J2KEncoder::end()
*/
for (auto & i: _queue) {
#ifdef DCPOMATIC_GROK
- if (Config::instance()->grok().get_value_or({}).enable) {
+ if (Config::instance()->grok().enable) {
if (!_context->scheduleCompress(i)){
LOG_GENERAL (N_("[%1] J2KEncoder thread pushes frame %2 back onto queue after failure"), thread_id(), i.index());
// handle error
diff --git a/src/lib/util.cc b/src/lib/util.cc
index df15e1abb..1e2f7d61f 100644
--- a/src/lib/util.cc
+++ b/src/lib/util.cc
@@ -1146,7 +1146,7 @@ setup_grok_library_path()
}
}
auto const grok = Config::instance()->grok();
- if (!grok || grok->binary_location.empty()) {
+ if (grok.binary_location.empty()) {
setenv("LD_LIRARY_PATH", old_path.c_str(), 1);
return;
}
@@ -1155,7 +1155,7 @@ setup_grok_library_path()
if (!new_path.empty()) {
new_path += ":";
}
- new_path += grok->binary_location.string();
+ new_path += grok.binary_location.string();
setenv("LD_LIBRARY_PATH", new_path.c_str(), 1);
}
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);
}