Move UTC offset for KDMs from the cinema to the point of KDM creation (#2300).

[dcpomatic.git] / src / lib / cinema.cc

diff --git a/src/lib/cinema.cc b/src/lib/cinema.cc
index 57780fa7b859cd456f47052fd9b5d9d9ac065175..7388dbc2fb890390a15ca75f9f5beead2f956a20 100644 (file)
--- a/src/lib/cinema.cc
+++ b/src/lib/cinema.cc
@@ -41,14 +41,6 @@ Cinema::Cinema (cxml::ConstNodePtr node)
        for (auto i: node->node_children("Email")) {
                emails.push_back (i->content ());
        }
-
-       if (node->optional_number_child<int>("UTCOffset")) {
-               _utc_offset_hour = node->number_child<int>("UTCOffset");
-       } else {
-               _utc_offset_hour = node->optional_number_child<int>("UTCOffsetHour").get_value_or (0);
-       }
-
-       _utc_offset_minute = node->optional_number_child<int>("UTCOffsetMinute").get_value_or (0);
 }
 
 /* This is necessary so that we can use shared_from_this in add_screen (which cannot be done from
@@ -73,9 +65,6 @@ Cinema::as_xml (xmlpp::Element* parent) const
 
        parent->add_child("Notes")->add_child_text (notes);
 
-       parent->add_child("UTCOffsetHour")->add_child_text (raw_convert<string> (_utc_offset_hour));
-       parent->add_child("UTCOffsetMinute")->add_child_text (raw_convert<string> (_utc_offset_minute));
-
        for (auto i: _screens) {
                i->as_xml (parent->add_child ("Screen"));
        }
@@ -91,19 +80,9 @@ Cinema::add_screen (shared_ptr<Screen> s)
 void
 Cinema::remove_screen (shared_ptr<Screen> s)
 {
-       _screens.remove (s);
+       auto iter = std::find(_screens.begin(), _screens.end(), s);
+       if (iter != _screens.end()) {
+               _screens.erase(iter);
+       }
 }
 
-void
-Cinema::set_utc_offset_hour (int h)
-{
-       DCPOMATIC_ASSERT (h >= -11 && h <= 12);
-       _utc_offset_hour = h;
-}
-
-void
-Cinema::set_utc_offset_minute (int m)
-{
-       DCPOMATIC_ASSERT (m >= 0 && m <= 59);
-       _utc_offset_minute = m;
-}