Slight hack to reduce the chance of config files looking different
[dcpomatic.git] / src / lib / config.cc
index e2425799d16f8253efb1fd6ba88b3a9fe5a50e30..23b06f2e40a7ac9fdeea84eccd05344473c76b64 100644 (file)
@@ -1,5 +1,5 @@
 /*
-    Copyright (C) 2012-2020 Carl Hetherington <cth@carlh.net>
+    Copyright (C) 2012-2021 Carl Hetherington <cth@carlh.net>
 
     This file is part of DCP-o-matic.
 
@@ -41,7 +41,6 @@
 #include <libxml++/libxml++.h>
 #include <boost/filesystem.hpp>
 #include <boost/algorithm/string.hpp>
-#include <boost/foreach.hpp>
 #include <boost/thread.hpp>
 #include <cstdlib>
 #include <fstream>
@@ -60,6 +59,7 @@ using std::remove;
 using std::exception;
 using std::cerr;
 using std::shared_ptr;
+using std::make_shared;
 using boost::optional;
 using std::dynamic_pointer_cast;
 using boost::algorithm::trim;
@@ -88,7 +88,7 @@ Config::set_defaults ()
        _use_any_servers = true;
        _servers.clear ();
        _only_servers_encode = false;
-       _tms_protocol = FILE_TRANSFER_PROTOCOL_SCP;
+       _tms_protocol = FileTransferProtocol::SCP;
        _tms_ip = "";
        _tms_path = ".";
        _tms_user = "";
@@ -107,7 +107,7 @@ Config::set_defaults ()
        _upload_after_make_dcp = false;
        _mail_server = "";
        _mail_port = 25;
-       _mail_protocol = EMAIL_PROTOCOL_AUTO;
+       _mail_protocol = EmailProtocol::AUTO;
        _mail_user = "";
        _mail_password = "";
        _kdm_from = "";
@@ -175,6 +175,7 @@ Config::set_defaults ()
        _player_playlist_directory = boost::none;
        _player_kdm_directory = boost::none;
        _audio_mapping = boost::none;
+       _minimum_frame_size = 65536;
 
        _allowed_dcp_frame_rates.clear ();
        _allowed_dcp_frame_rates.push_back (24);
@@ -199,15 +200,13 @@ Config::restore_defaults ()
 shared_ptr<dcp::CertificateChain>
 Config::create_certificate_chain ()
 {
-       return shared_ptr<dcp::CertificateChain> (
-               new dcp::CertificateChain (
-                       openssl_path(),
-                       "dcpomatic.com",
-                       "dcpomatic.com",
-                       ".dcpomatic.smpte-430-2.ROOT",
-                       ".dcpomatic.smpte-430-2.INTERMEDIATE",
-                       "CS.dcpomatic.smpte-430-2.LEAF"
-                       )
+       return make_shared<dcp::CertificateChain> (
+               openssl_path(),
+               "dcpomatic.com",
+               "dcpomatic.com",
+               ".dcpomatic.smpte-430-2.ROOT",
+               ".dcpomatic.smpte-430-2.INTERMEDIATE",
+               "CS.dcpomatic.smpte-430-2.LEAF"
                );
 }
 
@@ -234,7 +233,7 @@ try
        cxml::Document f ("Config");
        f.read_file (config_file ());
 
-       optional<int> version = f.optional_number_child<int> ("Version");
+       auto version = f.optional_number_child<int> ("Version");
        if (version && *version < _current_version) {
                /* Back up the old config before we re-write it in a back-incompatible way */
                backup ();
@@ -253,16 +252,16 @@ try
                _default_directory = boost::optional<boost::filesystem::path> ();
        }
 
-       boost::optional<int> b = f.optional_number_child<int> ("ServerPort");
+       auto b = f.optional_number_child<int> ("ServerPort");
        if (!b) {
                b = f.optional_number_child<int> ("ServerPortBase");
        }
        _server_port_base = b.get ();
 
-       boost::optional<bool> u = f.optional_bool_child ("UseAnyServers");
+       auto u = f.optional_bool_child ("UseAnyServers");
        _use_any_servers = u.get_value_or (true);
 
-       BOOST_FOREACH (cxml::ConstNodePtr i, f.node_children("Server")) {
+       for (auto i: f.node_children("Server")) {
                if (i->node_children("HostName").size() == 1) {
                        _servers.push_back (i->string_child ("HostName"));
                } else {
@@ -271,7 +270,7 @@ try
        }
 
        _only_servers_encode = f.optional_bool_child ("OnlyServersEncode").get_value_or (false);
-       _tms_protocol = static_cast<FileTransferProtocol>(f.optional_number_child<int>("TMSProtocol").get_value_or(static_cast<int>(FILE_TRANSFER_PROTOCOL_SCP)));
+       _tms_protocol = static_cast<FileTransferProtocol>(f.optional_number_child<int>("TMSProtocol").get_value_or(static_cast<int>(FileTransferProtocol::SCP)));
        _tms_ip = f.string_child ("TMSIP");
        _tms_path = f.string_child ("TMSPath");
        _tms_user = f.string_child ("TMSUser");
@@ -279,7 +278,7 @@ try
 
        _language = f.optional_string_child ("Language");
 
-       optional<string> c = f.optional_string_child ("DefaultContainer");
+       auto c = f.optional_string_child ("DefaultContainer");
        if (c) {
                _default_container = Ratio::from_id (c.get ());
        }
@@ -298,7 +297,7 @@ try
                _dcp_issuer = f.string_child ("DCPIssuer");
        }
 
-       optional<bool> up = f.optional_bool_child("UploadAfterMakeDCP");
+       auto up = f.optional_bool_child("UploadAfterMakeDCP");
        if (!up) {
                up = f.optional_bool_child("DefaultUploadAfterMakeDCP");
        }
@@ -333,13 +332,13 @@ try
                /* Make sure this matches the code in write_config */
                string const protocol = f.optional_string_child("MailProtocol").get_value_or("Auto");
                if (protocol == "Auto") {
-                       _mail_protocol = EMAIL_PROTOCOL_AUTO;
+                       _mail_protocol = EmailProtocol::AUTO;
                } else if (protocol == "Plain") {
-                       _mail_protocol = EMAIL_PROTOCOL_PLAIN;
+                       _mail_protocol = EmailProtocol::PLAIN;
                } else if (protocol == "STARTTLS") {
-                       _mail_protocol = EMAIL_PROTOCOL_STARTTLS;
+                       _mail_protocol = EmailProtocol::STARTTLS;
                } else if (protocol == "SSL") {
-                       _mail_protocol = EMAIL_PROTOCOL_SSL;
+                       _mail_protocol = EmailProtocol::SSL;
                }
        }
 
@@ -348,7 +347,7 @@ try
 
        _kdm_subject = f.optional_string_child ("KDMSubject").get_value_or (_("KDM delivery: $CPL_NAME"));
        _kdm_from = f.string_child ("KDMFrom");
-       BOOST_FOREACH (cxml::ConstNodePtr i, f.node_children("KDMCC")) {
+       for (auto i: f.node_children("KDMCC")) {
                if (!i->content().empty()) {
                        _kdm_cc.push_back (i->content ());
                }
@@ -359,7 +358,7 @@ try
        _notification_subject = f.optional_string_child("NotificationSubject").get_value_or(_("DCP-o-matic notification"));
        _notification_from = f.optional_string_child("NotificationFrom").get_value_or("");
        _notification_to = f.optional_string_child("NotificationTo").get_value_or("");
-       BOOST_FOREACH (cxml::ConstNodePtr i, f.node_children("NotificationCC")) {
+       for (auto i: f.node_children("NotificationCC")) {
                if (!i->content().empty()) {
                        _notification_cc.push_back (i->content ());
                }
@@ -384,19 +383,19 @@ try
        _win32_console = f.optional_bool_child ("Win32Console").get_value_or (false);
 #endif
 
-       BOOST_FOREACH (cxml::ConstNodePtr i, f.node_children("History")) {
+       for (auto i: f.node_children("History")) {
                _history.push_back (i->content ());
        }
 
-       BOOST_FOREACH (cxml::ConstNodePtr i, f.node_children("PlayerHistory")) {
+       for (auto i: f.node_children("PlayerHistory")) {
                _player_history.push_back (i->content ());
        }
 
-       cxml::NodePtr signer = f.optional_node_child ("Signer");
+       auto signer = f.optional_node_child ("Signer");
        if (signer) {
-               shared_ptr<dcp::CertificateChain> c (new dcp::CertificateChain ());
+               auto c = make_shared<dcp::CertificateChain>();
                /* Read the signing certificates and private key in from the config file */
-               BOOST_FOREACH (cxml::NodePtr i, signer->node_children ("Certificate")) {
+               for (auto i: signer->node_children ("Certificate")) {
                        c->add (dcp::Certificate (i->content ()));
                }
                c->set_key (signer->string_child ("PrivateKey"));
@@ -406,10 +405,10 @@ try
                _signer_chain = create_certificate_chain ();
        }
 
-       cxml::NodePtr decryption = f.optional_node_child ("Decryption");
+       auto decryption = f.optional_node_child ("Decryption");
        if (decryption) {
-               shared_ptr<dcp::CertificateChain> c (new dcp::CertificateChain ());
-               BOOST_FOREACH (cxml::NodePtr i, decryption->node_children ("Certificate")) {
+               auto c = make_shared<dcp::CertificateChain>();
+               for (auto i: decryption->node_children ("Certificate")) {
                        c->add (dcp::Certificate (i->content ()));
                }
                c->set_key (decryption->string_child ("PrivateKey"));
@@ -421,8 +420,8 @@ try
        /* These must be done before we call Bad as that might set one
           of the nags.
        */
-       BOOST_FOREACH (cxml::NodePtr i, f.node_children("Nagged")) {
-               int const id = i->number_attribute<int>("Id");
+       for (auto i: f.node_children("Nagged")) {
+               auto const id = i->number_attribute<int>("Id");
                if (id >= 0 && id < NAG_COUNT) {
                        _nagged[id] = raw_convert<int>(i->content());
                }
@@ -430,7 +429,7 @@ try
 
        optional<BadReason> bad;
 
-       BOOST_FOREACH (dcp::Certificate const & i, _signer_chain->unordered()) {
+       for (auto const& i: _signer_chain->unordered()) {
                if (i.has_utf8_strings()) {
                        bad = BAD_SIGNER_UTF8_STRINGS;
                }
@@ -445,7 +444,7 @@ try
        }
 
        if (bad) {
-               optional<bool> const remake = Bad(*bad);
+               auto const remake = Bad(*bad);
                if (remake && *remake) {
                        switch (*bad) {
                        case BAD_SIGNER_UTF8_STRINGS:
@@ -465,7 +464,7 @@ try
        } else {
                /* Old-style: one or more DKDM nodes */
                _dkdms.reset (new DKDMGroup ("root"));
-               BOOST_FOREACH (cxml::ConstNodePtr i, f.node_children("DKDM")) {
+               for (auto i: f.node_children("DKDM")) {
                        _dkdms->add (DKDMBase::read (i));
                }
        }
@@ -506,7 +505,7 @@ try
        _decode_reduction = f.optional_number_child<int>("DecodeReduction");
        _default_notify = f.optional_bool_child("DefaultNotify").get_value_or(false);
 
-       BOOST_FOREACH (cxml::NodePtr i, f.node_children("Notification")) {
+       for (auto i: f.node_children("Notification")) {
                int const id = i->number_attribute<int>("Id");
                if (id >= 0 && id < NOTIFICATION_COUNT) {
                        _notification[id] = raw_convert<int>(i->content());
@@ -520,7 +519,7 @@ try
        _gdc_username = f.optional_string_child("GDCUsername");
        _gdc_password = f.optional_string_child("GDCPassword");
 
-       optional<string> pm = f.optional_string_child("PlayerMode");
+       auto pm = f.optional_string_child("PlayerMode");
        if (pm && *pm == "window") {
                _player_mode = PLAYER_MODE_WINDOW;
        } else if (pm && *pm == "full") {
@@ -530,7 +529,7 @@ try
        }
 
        _image_display = f.optional_number_child<int>("ImageDisplay").get_value_or(0);
-       optional<string> vc = f.optional_string_child("VideoViewType");
+       auto vc = f.optional_string_child("VideoViewType");
        if (vc && *vc == "opengl") {
                _video_view_type = VIDEO_VIEW_OPENGL;
        } else if (vc && *vc == "simple") {
@@ -551,6 +550,8 @@ try
                _audio_mapping = AudioMapping (f.node_child("AudioMapping"), Film::current_state_version);
        }
 
+       _minimum_frame_size = f.optional_number_child<int>("MinimumFrameSize").get_value_or(65536);
+
        if (boost::filesystem::exists (_cinemas_file)) {
                cxml::Document f ("Cinemas");
                f.read_file (_cinemas_file);
@@ -602,7 +603,7 @@ void
 Config::write_config () const
 {
        xmlpp::Document doc;
-       xmlpp::Element* root = doc.create_root_node ("Config");
+       auto root = doc.create_root_node ("Config");
 
        /* [XML] Version The version number of the configuration file format. */
        root->add_child("Version")->add_child_text (raw_convert<string>(_current_version));
@@ -622,7 +623,7 @@ Config::write_config () const
        /* [XML] UseAnyServers 1 to broadcast to look for encoding servers to use, 0 to use only those configured. */
        root->add_child("UseAnyServers")->add_child_text (_use_any_servers ? "1" : "0");
 
-       BOOST_FOREACH (string i, _servers) {
+       for (auto i: _servers) {
                /* [XML:opt] Server IP address or hostname of an encoding server to use; you can use as many of these tags
                   as you like.
                */
@@ -664,7 +665,7 @@ Config::write_config () const
        root->add_child("DefaultDCPAudioChannels")->add_child_text (raw_convert<string> (_default_dcp_audio_channels));
        /* [XML] DCPIssuer Issuer text to write into CPL files. */
        root->add_child("DCPIssuer")->add_child_text (_dcp_issuer);
-       /* [XML] DCPIssuer Creator text to write into CPL files. */
+       /* [XML] DCPCreator Creator text to write into CPL files. */
        root->add_child("DCPCreator")->add_child_text (_dcp_creator);
        /* [XML] Company name to write into MXF files. */
        root->add_child("DCPCompanyName")->add_child_text (_dcp_company_name);
@@ -703,16 +704,16 @@ Config::write_config () const
        root->add_child("MailPort")->add_child_text (raw_convert<string> (_mail_port));
        /* [XML] MailProtocol Protocol to use on SMTP server (Auto, Plain, STARTTLS or SSL) */
        switch (_mail_protocol) {
-       case EMAIL_PROTOCOL_AUTO:
+       case EmailProtocol::AUTO:
                root->add_child("MailProtocol")->add_child_text("Auto");
                break;
-       case EMAIL_PROTOCOL_PLAIN:
+       case EmailProtocol::PLAIN:
                root->add_child("MailProtocol")->add_child_text("Plain");
                break;
-       case EMAIL_PROTOCOL_STARTTLS:
+       case EmailProtocol::STARTTLS:
                root->add_child("MailProtocol")->add_child_text("STARTTLS");
                break;
-       case EMAIL_PROTOCOL_SSL:
+       case EmailProtocol::SSL:
                root->add_child("MailProtocol")->add_child_text("SSL");
                break;
        }
@@ -725,7 +726,7 @@ Config::write_config () const
        root->add_child("KDMSubject")->add_child_text (_kdm_subject);
        /* [XML] KDMFrom From address to use for KDM emails. */
        root->add_child("KDMFrom")->add_child_text (_kdm_from);
-       BOOST_FOREACH (string i, _kdm_cc) {
+       for (auto i: _kdm_cc) {
                /* [XML] KDMCC CC address to use for KDM emails; you can use as many of these tags as you like. */
                root->add_child("KDMCC")->add_child_text (i);
        }
@@ -740,7 +741,7 @@ Config::write_config () const
        root->add_child("NotificationFrom")->add_child_text (_notification_from);
        /* [XML] NotificationFrom To address to use for notification emails. */
        root->add_child("NotificationTo")->add_child_text (_notification_to);
-       BOOST_FOREACH (string i, _notification_cc) {
+       for (auto i: _notification_cc) {
                /* [XML] NotificationCC CC address to use for notification emails; you can use as many of these tags as you like. */
                root->add_child("NotificationCC")->add_child_text (i);
        }
@@ -773,24 +774,29 @@ Config::write_config () const
        /* [XML] AutomaticAudioAnalysis 1 to run audio analysis automatically when audio content is added to the film, otherwise 0. */
        root->add_child("AutomaticAudioAnalysis")->add_child_text (_automatic_audio_analysis ? "1" : "0");
 #ifdef DCPOMATIC_WINDOWS
-       /* [XML] Win32Console 1 to open a console when running on Windows, otherwise 0. */
-       root->add_child("Win32Console")->add_child_text (_win32_console ? "1" : "0");
+       if (_win32_console) {
+               /* [XML] Win32Console 1 to open a console when running on Windows, otherwise 0.
+                * We only write this if it's true, which is a bit of a hack to allow unit tests to work
+                * more easily on Windows (without a platform-specific reference in config_write_utf8_test)
+                */
+               root->add_child("Win32Console")->add_child_text ("1");
+       }
 #endif
 
        /* [XML] Signer Certificate chain and private key to use when signing DCPs and KDMs.  Should contain <code>&lt;Certificate&gt;</code>
           tags in order and a <code>&lt;PrivateKey&gt;</code> tag all containing PEM-encoded certificates or private keys as appropriate.
        */
-       xmlpp::Element* signer = root->add_child ("Signer");
+       auto signer = root->add_child ("Signer");
        DCPOMATIC_ASSERT (_signer_chain);
-       BOOST_FOREACH (dcp::Certificate const & i, _signer_chain->unordered()) {
+       for (auto const& i: _signer_chain->unordered()) {
                signer->add_child("Certificate")->add_child_text (i.certificate (true));
        }
        signer->add_child("PrivateKey")->add_child_text (_signer_chain->key().get ());
 
        /* [XML] Decryption Certificate chain and private key to use when decrypting KDMs */
-       xmlpp::Element* decryption = root->add_child ("Decryption");
+       auto decryption = root->add_child ("Decryption");
        DCPOMATIC_ASSERT (_decryption_chain);
-       BOOST_FOREACH (dcp::Certificate const & i, _decryption_chain->unordered()) {
+       for (auto const& i: _decryption_chain->unordered()) {
                decryption->add_child("Certificate")->add_child_text (i.certificate (true));
        }
        decryption->add_child("PrivateKey")->add_child_text (_decryption_chain->key().get ());
@@ -798,14 +804,14 @@ Config::write_config () const
        /* [XML] History Filename of DCP to present in the <guilabel>File</guilabel> menu of the GUI; there can be more than one
           of these tags.
        */
-       BOOST_FOREACH (boost::filesystem::path i, _history) {
+       for (auto i: _history) {
                root->add_child("History")->add_child_text (i.string ());
        }
 
        /* [XML] History Filename of DCP to present in the <guilabel>File</guilabel> menu of the player; there can be more than one
           of these tags.
        */
-       BOOST_FOREACH (boost::filesystem::path i, _player_history) {
+       for (auto i: _player_history) {
                root->add_child("PlayerHistory")->add_child_text (i.string ());
        }
 
@@ -974,15 +980,16 @@ Config::write_config () const
        if (_audio_mapping) {
                _audio_mapping->as_xml (root->add_child("AudioMapping"));
        }
+       root->add_child("MinimumFrameSize")->add_child_text(raw_convert<string>(_minimum_frame_size));
 
        try {
-               string const s = doc.write_to_string_formatted ();
+               auto const s = doc.write_to_string_formatted ();
                boost::filesystem::path tmp (string(config_file().string()).append(".tmp"));
-               FILE* f = fopen_boost (tmp, "w");
+               auto f = fopen_boost (tmp, "w");
                if (!f) {
                        throw FileError (_("Could not open file for writing"), tmp);
                }
-               checked_fwrite (s.c_str(), s.length(), f, tmp);
+               checked_fwrite (s.c_str(), s.bytes(), f, tmp);
                fclose (f);
                boost::filesystem::remove (config_file());
                boost::filesystem::rename (tmp, config_file());
@@ -996,13 +1003,13 @@ Config::write_config () const
 
 template <class T>
 void
-write_file (string root_node, string node, string version, list<shared_ptr<T> > things, boost::filesystem::path file)
+write_file (string root_node, string node, string version, list<shared_ptr<T>> things, boost::filesystem::path file)
 {
        xmlpp::Document doc;
-       xmlpp::Element* root = doc.create_root_node (root_node);
+       auto root = doc.create_root_node (root_node);
        root->add_child("Version")->add_child_text(version);
 
-       BOOST_FOREACH (shared_ptr<T> i, things) {
+       for (auto i: things) {
                i->as_xml (root->add_child(node));
        }
 
@@ -1052,7 +1059,7 @@ Config::directory_or (optional<boost::filesystem::path> dir, boost::filesystem::
        }
 
        boost::system::error_code ec;
-       bool const e = boost::filesystem::exists (*dir, ec);
+       auto const e = boost::filesystem::exists (*dir, ec);
        if (ec || !e) {
                return a;
        }
@@ -1170,9 +1177,9 @@ Config::add_to_history_internal (vector<boost::filesystem::path>& h, boost::file
 void
 Config::clean_history_internal (vector<boost::filesystem::path>& h)
 {
-       vector<boost::filesystem::path> old = h;
+       auto old = h;
        h.clear ();
-       BOOST_FOREACH (boost::filesystem::path i, old) {
+       for (auto i: old) {
                try {
                        if (boost::filesystem::is_directory(i)) {
                                h.push_back (i);
@@ -1193,12 +1200,11 @@ void
 Config::read_cinemas (cxml::Document const & f)
 {
        _cinemas.clear ();
-       list<cxml::NodePtr> cin = f.node_children ("Cinema");
-       BOOST_FOREACH (cxml::ConstNodePtr i, f.node_children("Cinema")) {
+       for (auto i: f.node_children("Cinema")) {
                /* Slightly grotty two-part construction of Cinema here so that we can use
                   shared_from_this.
                */
-               shared_ptr<Cinema> cinema (new Cinema (i));
+               auto cinema = make_shared<Cinema>(i);
                cinema->read_screens (i);
                _cinemas.push_back (cinema);
        }
@@ -1228,8 +1234,7 @@ void
 Config::read_dkdm_recipients (cxml::Document const & f)
 {
        _dkdm_recipients.clear ();
-       list<cxml::NodePtr> cin = f.node_children ("DKDMRecipient");
-       BOOST_FOREACH (cxml::ConstNodePtr i, f.node_children("DKDMRecipient")) {
+       for (auto i: f.node_children("DKDMRecipient")) {
                _dkdm_recipients.push_back (shared_ptr<DKDMRecipient>(new DKDMRecipient(i)));
        }
 }
@@ -1264,12 +1269,12 @@ list<string>
 Config::templates () const
 {
        if (!boost::filesystem::exists (path ("templates"))) {
-               return list<string> ();
+               return {};
        }
 
        list<string> n;
-       for (boost::filesystem::directory_iterator i (path("templates")); i != boost::filesystem::directory_iterator(); ++i) {
-               n.push_back (i->path().filename().string());
+       for (auto const& i: boost::filesystem::directory_iterator(path("templates"))) {
+               n.push_back (i.path().filename().string());
        }
        return n;
 }
@@ -1303,7 +1308,7 @@ boost::filesystem::path
 Config::config_file ()
 {
        cxml::Document f ("Config");
-       boost::filesystem::path main = path("config.xml", false);
+       auto main = path("config.xml", false);
        if (!boost::filesystem::exists (main)) {
                /* It doesn't exist, so there can't be any links; just return it */
                return main;
@@ -1312,7 +1317,7 @@ Config::config_file ()
        /* See if there's a link */
        try {
                f.read_file (main);
-               optional<string> link = f.optional_string_child("Link");
+               auto link = f.optional_string_child("Link");
                if (link) {
                        return *link;
                }
@@ -1357,7 +1362,7 @@ Config::copy_and_link (boost::filesystem::path new_file) const
 bool
 Config::have_write_permission () const
 {
-       FILE* f = fopen_boost (config_file(), "r+");
+       auto f = fopen_boost (config_file(), "r+");
        if (!f) {
                return false;
        }
@@ -1380,14 +1385,14 @@ Config::audio_mapping (int output_channels)
                           Map so that Lt = L(-3dB) + Ls(-3dB) + C(-6dB) + Lfe(-10dB)
                           Rt = R(-3dB) + Rs(-3dB) + C(-6dB) + Lfe(-10dB)
                        */
-                       _audio_mapping->set (dcp::LEFT,   0, 1 / sqrt(2));  // L   -> Lt
-                       _audio_mapping->set (dcp::RIGHT,  1, 1 / sqrt(2));  // R   -> Rt
-                       _audio_mapping->set (dcp::CENTRE, 0, 1 / 2.0);      // C   -> Lt
-                       _audio_mapping->set (dcp::CENTRE, 1, 1 / 2.0);      // C   -> Rt
-                       _audio_mapping->set (dcp::LFE,    0, 1 / sqrt(10)); // Lfe -> Lt
-                       _audio_mapping->set (dcp::LFE,    1, 1 / sqrt(10)); // Lfe -> Rt
-                       _audio_mapping->set (dcp::LS,     0, 1 / sqrt(2));  // Ls  -> Lt
-                       _audio_mapping->set (dcp::RS,     1, 1 / sqrt(2));  // Rs  -> Rt
+                       _audio_mapping->set (dcp::Channel::LEFT,   0, 1 / sqrt(2));  // L   -> Lt
+                       _audio_mapping->set (dcp::Channel::RIGHT,  1, 1 / sqrt(2));  // R   -> Rt
+                       _audio_mapping->set (dcp::Channel::CENTRE, 0, 1 / 2.0);      // C   -> Lt
+                       _audio_mapping->set (dcp::Channel::CENTRE, 1, 1 / 2.0);      // C   -> Rt
+                       _audio_mapping->set (dcp::Channel::LFE,    0, 1 / sqrt(10)); // Lfe -> Lt
+                       _audio_mapping->set (dcp::Channel::LFE,    1, 1 / sqrt(10)); // Lfe -> Rt
+                       _audio_mapping->set (dcp::Channel::LS,     0, 1 / sqrt(2));  // Ls  -> Lt
+                       _audio_mapping->set (dcp::Channel::RS,     1, 1 / sqrt(2));  // Rs  -> Rt
                } else {
                        /* 1:1 mapping */
                        for (int i = 0; i < min (MAX_DCP_AUDIO_CHANNELS, output_channels); ++i) {
@@ -1410,7 +1415,7 @@ void
 Config::set_audio_mapping_to_default ()
 {
        DCPOMATIC_ASSERT (_audio_mapping);
-       int const ch = _audio_mapping->output_channels ();
+       auto const ch = _audio_mapping->output_channels ();
        _audio_mapping = boost::none;
        _audio_mapping = audio_mapping (ch);
        changed (AUDIO_MAPPING);