White space: dcp_panel.{cc,h}
authorCarl Hetherington <cth@carlh.net>
Fri, 25 Apr 2025 21:08:36 +0000 (23:08 +0200)
committerCarl Hetherington <cth@carlh.net>
Fri, 25 Apr 2025 21:08:36 +0000 (23:08 +0200)
src/wx/dcp_panel.cc
src/wx/dcp_panel.h

index aeddb2a8db8c3a43980c3b164acbe719a8c902f5..8c5b4ec7e32882fbbb33bf0ce89315ef3cefb205 100644 (file)
@@ -72,62 +72,62 @@ using dcp::locale_convert;
 
 
 DCPPanel::DCPPanel(wxNotebook* n, shared_ptr<Film> film, FilmViewer& viewer)
-       : _film (film)
-       , _viewer (viewer)
-       , _generally_sensitive (true)
+       : _film(film)
+       , _viewer(viewer)
+       , _generally_sensitive(true)
 {
-       _panel = new wxPanel (n);
-       _sizer = new wxBoxSizer (wxVERTICAL);
-       _panel->SetSizer (_sizer);
+       _panel = new wxPanel(n);
+       _sizer = new wxBoxSizer(wxVERTICAL);
+       _panel->SetSizer(_sizer);
 
-       _grid = new wxGridBagSizer (DCPOMATIC_SIZER_X_GAP, DCPOMATIC_SIZER_Y_GAP);
-       _sizer->Add (_grid, 0, wxEXPAND | wxALL, 8);
+       _grid = new wxGridBagSizer(DCPOMATIC_SIZER_X_GAP, DCPOMATIC_SIZER_Y_GAP);
+       _sizer->Add(_grid, 0, wxEXPAND | wxALL, 8);
 
-       _name_label = create_label (_panel, _("Name"), true);
-       _name = new wxTextCtrl (_panel, wxID_ANY);
+       _name_label = create_label(_panel, _("Name"), true);
+       _name = new wxTextCtrl(_panel, wxID_ANY);
        FocusManager::instance()->add(_name);
 
-       _use_isdcf_name = new CheckBox (_panel, _("Use ISDCF name"));
-       _copy_isdcf_name_button = new Button (_panel, _("Copy as name"));
+       _use_isdcf_name = new CheckBox(_panel, _("Use ISDCF name"));
+       _copy_isdcf_name_button = new Button(_panel, _("Copy as name"));
 
        /* wxST_ELLIPSIZE_MIDDLE works around a bug in GTK2 and/or wxWidgets, see
           http://trac.wxwidgets.org/ticket/12539
        */
-       _dcp_name = new StaticText (
+       _dcp_name = new StaticText(
                _panel, {}, wxDefaultPosition, wxDefaultSize,
                wxALIGN_CENTRE_HORIZONTAL | wxST_NO_AUTORESIZE | wxST_ELLIPSIZE_MIDDLE
                );
 
-       _dcp_content_type_label = create_label (_panel, _("Content Type"), true);
+       _dcp_content_type_label = create_label(_panel, _("Content Type"), true);
        _dcp_content_type = new Choice(_panel);
 
-       _encrypted = new CheckBox (_panel, _("Encrypted"));
+       _encrypted = new CheckBox(_panel, _("Encrypted"));
 
-        wxClientDC dc (_panel);
+        wxClientDC dc(_panel);
         auto size = dc.GetTextExtent(char_to_wx("GGGGGGGG..."));
-        size.SetHeight (-1);
+        size.SetHeight(-1);
 
-       _standard_label = create_label (_panel, _("Standard"), true);
+       _standard_label = create_label(_panel, _("Standard"), true);
        _standard = new Choice(_panel);
 
-       _markers = new Button (_panel, _("Markers..."));
-       _metadata = new Button (_panel, _("Metadata..."));
+       _markers = new Button(_panel, _("Markers..."));
+       _metadata = new Button(_panel, _("Metadata..."));
        _reels = new Button(_panel, _("Reels..."));
 
-       _notebook = new wxNotebook (_panel, wxID_ANY);
-       _sizer->Add (_notebook, 1, wxEXPAND | wxTOP, 6);
+       _notebook = new wxNotebook(_panel, wxID_ANY);
+       _sizer->Add(_notebook, 1, wxEXPAND | wxTOP, 6);
 
-       _notebook->AddPage (make_video_panel (), _("Video"), false);
-       _notebook->AddPage (make_audio_panel (), _("Audio"), false);
+       _notebook->AddPage(make_video_panel(), _("Video"), false);
+       _notebook->AddPage(make_audio_panel(), _("Audio"), false);
 
-       _name->Bind                  (wxEVT_TEXT,     boost::bind(&DCPPanel::name_changed, this));
+       _name->Bind(wxEVT_TEXT, boost::bind(&DCPPanel::name_changed, this));
        _use_isdcf_name->bind(&DCPPanel::use_isdcf_name_toggled, this);
-       _copy_isdcf_name_button->Bind(wxEVT_BUTTON,   boost::bind(&DCPPanel::copy_isdcf_name_button_clicked, this));
-       _dcp_content_type->Bind      (wxEVT_CHOICE,   boost::bind(&DCPPanel::dcp_content_type_changed, this));
+       _copy_isdcf_name_button->Bind(wxEVT_BUTTON, boost::bind(&DCPPanel::copy_isdcf_name_button_clicked, this));
+       _dcp_content_type->Bind(wxEVT_CHOICE, boost::bind(&DCPPanel::dcp_content_type_changed, this));
        _encrypted->bind(&DCPPanel::encrypted_toggled, this);
-       _standard->Bind              (wxEVT_CHOICE,   boost::bind(&DCPPanel::standard_changed, this));
-       _markers->Bind               (wxEVT_BUTTON,   boost::bind(&DCPPanel::markers_clicked, this));
-       _metadata->Bind              (wxEVT_BUTTON,   boost::bind(&DCPPanel::metadata_clicked, this));
+       _standard->Bind(wxEVT_CHOICE, boost::bind(&DCPPanel::standard_changed, this));
+       _markers->Bind(wxEVT_BUTTON, boost::bind(&DCPPanel::markers_clicked, this));
+       _metadata->Bind(wxEVT_BUTTON, boost::bind(&DCPPanel::metadata_clicked, this));
        _reels->Bind(wxEVT_BUTTON, boost::bind(&DCPPanel::reels_clicked, this));
 
        for (auto i: DCPContentType::all()) {
@@ -137,9 +137,9 @@ DCPPanel::DCPPanel(wxNotebook* n, shared_ptr<Film> film, FilmViewer& viewer)
        update_standards();
        _standard->SetToolTip(_("The standard that the DCP should use.  Interop is older, and SMPTE is the newer (current) standard.  If in doubt, choose 'SMPTE'"));
 
-       Config::instance()->Changed.connect (boost::bind(&DCPPanel::config_changed, this, _1));
+       Config::instance()->Changed.connect(boost::bind(&DCPPanel::config_changed, this, _1));
 
-       add_to_grid ();
+       add_to_grid();
 }
 
 
@@ -175,7 +175,7 @@ DCPPanel::set_standard()
 
 
 void
-DCPPanel::standard_changed ()
+DCPPanel::standard_changed()
 {
        if (!_film || !_standard->get()) {
                return;
@@ -206,14 +206,14 @@ DCPPanel::standard_changed ()
 
 
 void
-DCPPanel::add_to_grid ()
+DCPPanel::add_to_grid()
 {
        int r = 0;
 
-       auto name_sizer = new wxBoxSizer (wxHORIZONTAL);
-       name_sizer->Add (_name_label, 0, wxRIGHT | wxALIGN_CENTER_VERTICAL, DCPOMATIC_SIZER_X_GAP);
-       name_sizer->Add (_name, 1, wxRIGHT | wxALIGN_CENTER_VERTICAL, DCPOMATIC_SIZER_X_GAP);
-       _grid->Add (name_sizer, wxGBPosition(r, 0), wxGBSpan(1, 2), wxRIGHT | wxEXPAND, DCPOMATIC_DIALOG_BORDER);
+       auto name_sizer = new wxBoxSizer(wxHORIZONTAL);
+       name_sizer->Add(_name_label, 0, wxRIGHT | wxALIGN_CENTER_VERTICAL, DCPOMATIC_SIZER_X_GAP);
+       name_sizer->Add(_name, 1, wxRIGHT | wxALIGN_CENTER_VERTICAL, DCPOMATIC_SIZER_X_GAP);
+       _grid->Add(name_sizer, wxGBPosition(r, 0), wxGBSpan(1, 2), wxRIGHT | wxEXPAND, DCPOMATIC_DIALOG_BORDER);
        ++r;
 
        int flags = wxALIGN_CENTER_VERTICAL;
@@ -221,39 +221,39 @@ DCPPanel::add_to_grid ()
        flags |= wxALIGN_RIGHT;
 #endif
 
-       _grid->Add (_use_isdcf_name, wxGBPosition(r, 0), wxDefaultSpan, flags);
+       _grid->Add(_use_isdcf_name, wxGBPosition(r, 0), wxDefaultSpan, flags);
        {
-               auto s = new wxBoxSizer (wxHORIZONTAL);
-               s->Add (_copy_isdcf_name_button, 0, wxLEFT, DCPOMATIC_SIZER_X_GAP);
-               _grid->Add (s, wxGBPosition(r, 1), wxDefaultSpan, wxEXPAND | wxBOTTOM, DCPOMATIC_CHECKBOX_BOTTOM_PAD);
+               auto s = new wxBoxSizer(wxHORIZONTAL);
+               s->Add(_copy_isdcf_name_button, 0, wxLEFT, DCPOMATIC_SIZER_X_GAP);
+               _grid->Add(s, wxGBPosition(r, 1), wxDefaultSpan, wxEXPAND | wxBOTTOM, DCPOMATIC_CHECKBOX_BOTTOM_PAD);
        }
        ++r;
 
-       _grid->Add (_dcp_name, wxGBPosition(r, 0), wxGBSpan(1, 2), wxALIGN_CENTER_VERTICAL | wxEXPAND);
+       _grid->Add(_dcp_name, wxGBPosition(r, 0), wxGBSpan(1, 2), wxALIGN_CENTER_VERTICAL | wxEXPAND);
        ++r;
 
-       add_label_to_sizer (_grid, _dcp_content_type_label, true, wxGBPosition(r, 0));
-       _grid->Add (_dcp_content_type, wxGBPosition(r, 1));
+       add_label_to_sizer(_grid, _dcp_content_type_label, true, wxGBPosition(r, 0));
+       _grid->Add(_dcp_content_type, wxGBPosition(r, 1));
        ++r;
 
-       _grid->Add (_encrypted, wxGBPosition(r, 0), wxGBSpan(1, 2));
+       _grid->Add(_encrypted, wxGBPosition(r, 0), wxGBSpan(1, 2));
        ++r;
 
-       add_label_to_sizer (_grid, _standard_label, true, wxGBPosition(r, 0));
-       _grid->Add (_standard, wxGBPosition(r, 1), wxDefaultSpan, wxALIGN_CENTER_VERTICAL);
+       add_label_to_sizer(_grid, _standard_label, true, wxGBPosition(r, 0));
+       _grid->Add(_standard, wxGBPosition(r, 1), wxDefaultSpan, wxALIGN_CENTER_VERTICAL);
        ++r;
 
-       auto extra = new wxBoxSizer (wxHORIZONTAL);
-       extra->Add (_markers, 1, wxRIGHT, DCPOMATIC_SIZER_X_GAP);
-       extra->Add (_metadata, 1, wxRIGHT, DCPOMATIC_SIZER_X_GAP);
+       auto extra = new wxBoxSizer(wxHORIZONTAL);
+       extra->Add(_markers, 1, wxRIGHT, DCPOMATIC_SIZER_X_GAP);
+       extra->Add(_metadata, 1, wxRIGHT, DCPOMATIC_SIZER_X_GAP);
        extra->Add(_reels, 1, wxRIGHT, DCPOMATIC_SIZER_X_GAP);
-       _grid->Add (extra, wxGBPosition(r, 0), wxGBSpan(1, 2));
+       _grid->Add(extra, wxGBPosition(r, 0), wxGBSpan(1, 2));
        ++r;
 }
 
 
 void
-DCPPanel::name_changed ()
+DCPPanel::name_changed()
 {
        if (!_film) {
                return;
@@ -275,25 +275,25 @@ DCPPanel::video_bit_rate_changed()
 
 
 void
-DCPPanel::encrypted_toggled ()
+DCPPanel::encrypted_toggled()
 {
        if (!_film) {
                return;
        }
 
-       _film->set_encrypted (_encrypted->GetValue());
+       _film->set_encrypted(_encrypted->GetValue());
 }
 
 
 /** Called when the frame rate choice widget has been changed */
 void
-DCPPanel::frame_rate_choice_changed ()
+DCPPanel::frame_rate_choice_changed()
 {
        if (!_film || !_frame_rate_choice->get()) {
                return;
        }
 
-       _film->set_video_frame_rate (
+       _film->set_video_frame_rate(
                boost::lexical_cast<int>(
                        wx_to_std(_frame_rate_choice->GetString(*_frame_rate_choice->get()))
                        ),
@@ -304,18 +304,18 @@ DCPPanel::frame_rate_choice_changed ()
 
 /** Called when the frame rate spin widget has been changed */
 void
-DCPPanel::frame_rate_spin_changed ()
+DCPPanel::frame_rate_spin_changed()
 {
        if (!_film) {
                return;
        }
 
-       _film->set_video_frame_rate (_frame_rate_spin->GetValue());
+       _film->set_video_frame_rate(_frame_rate_spin->GetValue());
 }
 
 
 void
-DCPPanel::audio_channels_changed ()
+DCPPanel::audio_channels_changed()
 {
        if (!_film) {
                return;
@@ -326,7 +326,7 @@ DCPPanel::audio_channels_changed ()
 
 
 void
-DCPPanel::resolution_changed ()
+DCPPanel::resolution_changed()
 {
        if (!_film || !_resolution->get()) {
                return;
@@ -337,7 +337,7 @@ DCPPanel::resolution_changed ()
 
 
 void
-DCPPanel::markers_clicked ()
+DCPPanel::markers_clicked()
 {
        _markers_dialog.reset(_panel, _film, _viewer);
        _markers_dialog->Show();
@@ -345,16 +345,16 @@ DCPPanel::markers_clicked ()
 
 
 void
-DCPPanel::metadata_clicked ()
+DCPPanel::metadata_clicked()
 {
        if (_film->interop()) {
                _interop_metadata_dialog.reset(_panel, _film);
-               _interop_metadata_dialog->setup ();
-               _interop_metadata_dialog->Show ();
+               _interop_metadata_dialog->setup();
+               _interop_metadata_dialog->Show();
        } else {
                _smpte_metadata_dialog.reset(_panel, _film);
-               _smpte_metadata_dialog->setup ();
-               _smpte_metadata_dialog->Show ();
+               _smpte_metadata_dialog->setup();
+               _smpte_metadata_dialog->Show();
        }
 }
 
@@ -374,42 +374,42 @@ DCPPanel::film_changed(FilmProperty p)
        case FilmProperty::NONE:
                break;
        case FilmProperty::CONTAINER:
-               setup_container ();
+               setup_container();
                break;
        case FilmProperty::NAME:
-               checked_set (_name, _film->name());
-               setup_dcp_name ();
+               checked_set(_name, _film->name());
+               setup_dcp_name();
                break;
        case FilmProperty::DCP_CONTENT_TYPE:
        {
                auto index = DCPContentType::as_index(_film->dcp_content_type());
-               DCPOMATIC_ASSERT (index);
-               checked_set (_dcp_content_type, *index);
-               setup_dcp_name ();
+               DCPOMATIC_ASSERT(index);
+               checked_set(_dcp_content_type, *index);
+               setup_dcp_name();
                break;
        }
        case FilmProperty::ENCRYPTED:
-               checked_set (_encrypted, _film->encrypted ());
+               checked_set(_encrypted, _film->encrypted());
                break;
        case FilmProperty::RESOLUTION:
-               checked_set (_resolution, _film->resolution() == Resolution::TWO_K ? 0 : 1);
-               setup_container ();
-               setup_dcp_name ();
+               checked_set(_resolution, _film->resolution() == Resolution::TWO_K ? 0 : 1);
+               setup_container();
+               setup_dcp_name();
                break;
        case FilmProperty::VIDEO_BIT_RATE:
                checked_set(_video_bit_rate, _film->video_bit_rate(_film->video_encoding()) / 1000000);
                break;
        case FilmProperty::USE_ISDCF_NAME:
        {
-               checked_set (_use_isdcf_name, _film->use_isdcf_name());
+               checked_set(_use_isdcf_name, _film->use_isdcf_name());
                if (_film->use_isdcf_name()) {
                        /* We are going back to using an ISDCF name.  Remove anything after a _ in the current name,
                           in case the user has clicked 'Copy as name' then re-ticked 'Use ISDCF name' (#1513).
                        */
-                       string const name = _film->name ();
+                       string const name = _film->name();
                        _film->set_name(name.substr(0, name.find("_")));
                }
-               setup_dcp_name ();
+               setup_dcp_name();
                break;
        }
        case FilmProperty::VIDEO_FRAME_RATE:
@@ -417,42 +417,42 @@ DCPPanel::film_changed(FilmProperty p)
                bool done = false;
                for (unsigned int i = 0; i < _frame_rate_choice->GetCount(); ++i) {
                        if (wx_to_std(_frame_rate_choice->GetString(i)) == boost::lexical_cast<string>(_film->video_frame_rate())) {
-                               checked_set (_frame_rate_choice, i);
+                               checked_set(_frame_rate_choice, i);
                                done = true;
                                break;
                        }
                }
 
                if (!done) {
-                       checked_set (_frame_rate_choice, -1);
+                       checked_set(_frame_rate_choice, -1);
                }
 
-               checked_set (_frame_rate_spin, _film->video_frame_rate ());
+               checked_set(_frame_rate_spin, _film->video_frame_rate());
 
-               _best_frame_rate->Enable (_film->best_video_frame_rate () != _film->video_frame_rate ());
-               setup_dcp_name ();
+               _best_frame_rate->Enable(_film->best_video_frame_rate() != _film->video_frame_rate());
+               setup_dcp_name();
                break;
        }
        case FilmProperty::AUDIO_CHANNELS:
                if (_film->audio_channels() < minimum_allowed_audio_channels()) {
-                       _film->set_audio_channels (minimum_allowed_audio_channels());
+                       _film->set_audio_channels(minimum_allowed_audio_channels());
                } else {
-                       checked_set (_audio_channels, locale_convert<string>(max(minimum_allowed_audio_channels(), _film->audio_channels())));
-                       setup_dcp_name ();
+                       checked_set(_audio_channels, locale_convert<string>(max(minimum_allowed_audio_channels(), _film->audio_channels())));
+                       setup_dcp_name();
                }
                break;
        case FilmProperty::THREE_D:
-               checked_set (_three_d, _film->three_d());
-               setup_dcp_name ();
+               checked_set(_three_d, _film->three_d());
+               setup_dcp_name();
                break;
        case FilmProperty::REENCODE_J2K:
-               checked_set (_reencode_j2k, _film->reencode_j2k());
+               checked_set(_reencode_j2k, _film->reencode_j2k());
                break;
        case FilmProperty::INTEROP:
                update_standards();
                set_standard();
-               setup_dcp_name ();
-               _markers->Enable (!_film->interop());
+               setup_dcp_name();
+               _markers->Enable(!_film->interop());
                break;
        case FilmProperty::VIDEO_ENCODING:
                set_standard();
@@ -466,16 +466,16 @@ DCPPanel::film_changed(FilmProperty p)
                break;
        case FilmProperty::AUDIO_PROCESSOR:
                if (_film->audio_processor()) {
-                       checked_set (_audio_processor, _film->audio_processor()->id());
+                       checked_set(_audio_processor, _film->audio_processor()->id());
                } else {
-                       checked_set (_audio_processor, 0);
+                       checked_set(_audio_processor, 0);
                }
-               setup_audio_channels_choice (_audio_channels, minimum_allowed_audio_channels());
-               film_changed (FilmProperty::AUDIO_CHANNELS);
+               setup_audio_channels_choice(_audio_channels, minimum_allowed_audio_channels());
+               film_changed(FilmProperty::AUDIO_CHANNELS);
                break;
        case FilmProperty::CONTENT:
-               setup_dcp_name ();
-               setup_sensitivity ();
+               setup_dcp_name();
+               setup_sensitivity();
                /* Maybe we now have ATMOS content which changes our minimum_allowed_audio_channels */
                setup_audio_channels_choice(_audio_channels, minimum_allowed_audio_channels());
                film_changed(FilmProperty::AUDIO_CHANNELS);
@@ -483,16 +483,16 @@ DCPPanel::film_changed(FilmProperty p)
        case FilmProperty::AUDIO_LANGUAGE:
        {
                auto al = _film->audio_language();
-               checked_set (_enable_audio_language, static_cast<bool>(al));
+               checked_set(_enable_audio_language, static_cast<bool>(al));
                checked_set(_audio_language, al ? std_to_wx(al->as_string()) : wxString{});
-               setup_dcp_name ();
-               setup_sensitivity ();
+               setup_dcp_name();
+               setup_sensitivity();
                _audio_panel_sizer->Layout();
                break;
        }
        case FilmProperty::AUDIO_FRAME_RATE:
                if (_audio_sample_rate) {
-                       checked_set (_audio_sample_rate, _film->audio_frame_rate() == 48000 ? 0 : 1);
+                       checked_set(_audio_sample_rate, _film->audio_frame_rate() == 48000 ? 0 : 1);
                }
                break;
        case FilmProperty::CONTENT_VERSIONS:
@@ -508,7 +508,7 @@ DCPPanel::film_changed(FilmProperty p)
        case FilmProperty::CHAIN:
        case FilmProperty::LUMINANCE:
        case FilmProperty::TERRITORY_TYPE:
-               setup_dcp_name ();
+               setup_dcp_name();
                break;
        default:
                break;
@@ -517,7 +517,7 @@ DCPPanel::film_changed(FilmProperty p)
 
 
 void
-DCPPanel::film_content_changed (int property)
+DCPPanel::film_content_changed(int property)
 {
        if (property == AudioContentProperty::STREAMS ||
            property == TextContentProperty::USE ||
@@ -533,14 +533,14 @@ DCPPanel::film_content_changed (int property)
            property == DCPContentProperty::REFERENCE_VIDEO ||
            property == DCPContentProperty::REFERENCE_AUDIO ||
            property == DCPContentProperty::REFERENCE_TEXT) {
-               setup_dcp_name ();
-               setup_sensitivity ();
+               setup_dcp_name();
+               setup_sensitivity();
        }
 }
 
 
 void
-DCPPanel::setup_container ()
+DCPPanel::setup_container()
 {
        auto ratios = Ratio::containers();
        if (std::find(ratios.begin(), ratios.end(), _film->container()) == ratios.end()) {
@@ -558,10 +558,10 @@ DCPPanel::setup_container ()
        DCPOMATIC_ASSERT(iter != ratios.end());
 
        checked_set(_container, iter - ratios.begin());
-       auto const size = fit_ratio_within(_film->container()->ratio(), _film->full_frame ());
+       auto const size = fit_ratio_within(_film->container()->ratio(), _film->full_frame());
        checked_set(_container_size, wxString::Format(char_to_wx("%dx%d"), size.width, size.height));
 
-       setup_dcp_name ();
+       setup_dcp_name();
 
        _video_grid->Layout();
 }
@@ -569,14 +569,14 @@ DCPPanel::setup_container ()
 
 /** Called when the container widget has been changed */
 void
-DCPPanel::container_changed ()
+DCPPanel::container_changed()
 {
        if (!_film) {
                return;
        }
 
        if (auto const container = _container->get()) {
-               auto ratios = Ratio::containers ();
+               auto ratios = Ratio::containers();
                DCPOMATIC_ASSERT(*container < int(ratios.size()));
                _film->set_container(ratios[*container]);
        }
@@ -585,7 +585,7 @@ DCPPanel::container_changed ()
 
 /** Called when the DCP content type widget has been changed */
 void
-DCPPanel::dcp_content_type_changed ()
+DCPPanel::dcp_content_type_changed()
 {
        if (!_film) {
                return;
@@ -598,7 +598,7 @@ DCPPanel::dcp_content_type_changed ()
 
 
 void
-DCPPanel::set_film (shared_ptr<Film> film)
+DCPPanel::set_film(shared_ptr<Film> film)
 {
        /* We are changing film, so destroy any dialogs for the old one */
        _audio_dialog.reset();
@@ -611,7 +611,7 @@ DCPPanel::set_film (shared_ptr<Film> film)
        if (!_film) {
                /* Really should all the film_changed below but this might be enough */
                checked_set(_dcp_name, wxString{});
-               set_general_sensitivity (false);
+               set_general_sensitivity(false);
                return;
        }
 
@@ -643,46 +643,46 @@ DCPPanel::set_film (shared_ptr<Film> film)
 
 
 void
-DCPPanel::set_general_sensitivity (bool s)
+DCPPanel::set_general_sensitivity(bool s)
 {
        _generally_sensitive = s;
-       setup_sensitivity ();
+       setup_sensitivity();
 }
 
 
 void
-DCPPanel::setup_sensitivity ()
+DCPPanel::setup_sensitivity()
 {
        auto const mpeg2 = _film && _film->video_encoding() == VideoEncoding::MPEG2;
 
-       _name->Enable                   (_generally_sensitive);
-       _use_isdcf_name->Enable         (_generally_sensitive);
-       _dcp_content_type->Enable       (_generally_sensitive);
-       _copy_isdcf_name_button->Enable (_generally_sensitive);
-       _enable_audio_language->Enable  (_generally_sensitive);
-       _audio_language->Enable         (_enable_audio_language->GetValue());
-       _edit_audio_language->Enable    (_enable_audio_language->GetValue());
-       _encrypted->Enable              (_generally_sensitive);
-       _markers->Enable                (_generally_sensitive && _film && !_film->interop());
-       _metadata->Enable               (_generally_sensitive);
-       _reels->Enable                  (_generally_sensitive && _film);
-       _frame_rate_choice->Enable      (_generally_sensitive && _film && !_film->references_dcp_video() && !_film->contains_atmos_content());
-       _frame_rate_spin->Enable        (_generally_sensitive && _film && !_film->references_dcp_video() && !_film->contains_atmos_content());
-       _audio_channels->Enable         (_generally_sensitive && _film && !_film->references_dcp_audio());
-       _audio_processor->Enable        (_generally_sensitive && _film && !_film->references_dcp_audio());
-       _video_bit_rate->Enable         (_generally_sensitive && _film && !_film->references_dcp_video());
-       _container->Enable              (_generally_sensitive && _film && !_film->references_dcp_video() && !mpeg2);
-       _best_frame_rate->Enable (
+       _name->Enable                  (_generally_sensitive);
+       _use_isdcf_name->Enable        (_generally_sensitive);
+       _dcp_content_type->Enable      (_generally_sensitive);
+       _copy_isdcf_name_button->Enable(_generally_sensitive);
+       _enable_audio_language->Enable (_generally_sensitive);
+       _audio_language->Enable        (_enable_audio_language->GetValue());
+       _edit_audio_language->Enable   (_enable_audio_language->GetValue());
+       _encrypted->Enable             (_generally_sensitive);
+       _markers->Enable               (_generally_sensitive && _film && !_film->interop());
+       _metadata->Enable              (_generally_sensitive);
+       _reels->Enable                 (_generally_sensitive && _film);
+       _frame_rate_choice->Enable     (_generally_sensitive && _film && !_film->references_dcp_video() && !_film->contains_atmos_content());
+       _frame_rate_spin->Enable       (_generally_sensitive && _film && !_film->references_dcp_video() && !_film->contains_atmos_content());
+       _audio_channels->Enable        (_generally_sensitive && _film && !_film->references_dcp_audio());
+       _audio_processor->Enable       (_generally_sensitive && _film && !_film->references_dcp_audio());
+       _video_bit_rate->Enable        (_generally_sensitive && _film && !_film->references_dcp_video());
+       _container->Enable             (_generally_sensitive && _film && !_film->references_dcp_video() && !mpeg2);
+       _best_frame_rate->Enable(
                _generally_sensitive &&
                _film &&
-               _film->best_video_frame_rate () != _film->video_frame_rate() &&
+               _film->best_video_frame_rate() != _film->video_frame_rate() &&
                !_film->references_dcp_video() &&
                !_film->contains_atmos_content()
                );
-       _resolution->Enable             (_generally_sensitive && _film && !_film->references_dcp_video() && !mpeg2);
-       _three_d->Enable                (_generally_sensitive && _film && !_film->references_dcp_video() && !mpeg2);
+       _resolution->Enable            (_generally_sensitive && _film && !_film->references_dcp_video() && !mpeg2);
+       _three_d->Enable               (_generally_sensitive && _film && !_film->references_dcp_video() && !mpeg2);
 
-       _standard->Enable (
+       _standard->Enable(
                _generally_sensitive &&
                _film &&
                !_film->references_dcp_video() &&
@@ -690,76 +690,76 @@ DCPPanel::setup_sensitivity ()
                !_film->contains_atmos_content()
                );
 
-       _reencode_j2k->Enable           (_generally_sensitive && _film);
-       _show_audio->Enable             (_generally_sensitive && _film);
+       _reencode_j2k->Enable          (_generally_sensitive && _film);
+       _show_audio->Enable            (_generally_sensitive && _film);
 }
 
 
 void
-DCPPanel::use_isdcf_name_toggled ()
+DCPPanel::use_isdcf_name_toggled()
 {
        if (!_film) {
                return;
        }
 
-       _film->set_use_isdcf_name (_use_isdcf_name->GetValue());
+       _film->set_use_isdcf_name(_use_isdcf_name->GetValue());
 }
 
 void
-DCPPanel::setup_dcp_name ()
+DCPPanel::setup_dcp_name()
 {
        if (!_film) {
                return;
        }
 
-       _dcp_name->SetLabel (std_to_wx(_film->dcp_name(true)));
-       _dcp_name->SetToolTip (std_to_wx(_film->dcp_name(true)));
+       _dcp_name->SetLabel(std_to_wx(_film->dcp_name(true)));
+       _dcp_name->SetToolTip(std_to_wx(_film->dcp_name(true)));
 }
 
 
 void
-DCPPanel::best_frame_rate_clicked ()
+DCPPanel::best_frame_rate_clicked()
 {
        if (!_film) {
                return;
        }
 
-       _film->set_video_frame_rate (_film->best_video_frame_rate());
+       _film->set_video_frame_rate(_film->best_video_frame_rate());
 }
 
 
 void
-DCPPanel::three_d_changed ()
+DCPPanel::three_d_changed()
 {
        if (!_film) {
                return;
        }
 
-       _film->set_three_d (_three_d->GetValue());
+       _film->set_three_d(_three_d->GetValue());
 }
 
 
 void
-DCPPanel::reencode_j2k_changed ()
+DCPPanel::reencode_j2k_changed()
 {
        if (!_film) {
                return;
        }
 
-       _film->set_reencode_j2k (_reencode_j2k->GetValue());
+       _film->set_reencode_j2k(_reencode_j2k->GetValue());
 }
 
 
 void
-DCPPanel::config_changed (Config::Property p)
+DCPPanel::config_changed(Config::Property p)
 {
        VideoEncoding const encoding = _film ? _film->video_encoding() : VideoEncoding::JPEG2000;
        _video_bit_rate->SetRange(1, Config::instance()->maximum_video_bit_rate(encoding) / 1000000);
-       setup_frame_rate_widget ();
+       setup_frame_rate_widget();
 
        if (p == Config::SHOW_EXPERIMENTAL_AUDIO_PROCESSORS) {
-               _audio_processor->Clear ();
-               add_audio_processors ();
+               _audio_processor->Clear();
+               add_audio_processors();
                if (_film) {
                        film_changed(FilmProperty::AUDIO_PROCESSOR);
                }
@@ -778,56 +778,56 @@ DCPPanel::config_changed (Config::Property p)
 
 
 void
-DCPPanel::setup_frame_rate_widget ()
+DCPPanel::setup_frame_rate_widget()
 {
        if (Config::instance()->allow_any_dcp_frame_rate()) {
-               _frame_rate_choice->Hide ();
-               _frame_rate_spin->Show ();
+               _frame_rate_choice->Hide();
+               _frame_rate_spin->Show();
        } else {
-               _frame_rate_choice->Show ();
-               _frame_rate_spin->Hide ();
+               _frame_rate_choice->Show();
+               _frame_rate_spin->Hide();
        }
        _frame_rate_sizer->Layout();
 }
 
 
 wxPanel *
-DCPPanel::make_video_panel ()
+DCPPanel::make_video_panel()
 {
-       auto panel = new wxPanel (_notebook);
-       auto sizer = new wxBoxSizer (wxVERTICAL);
-       _video_grid = new wxGridBagSizer (DCPOMATIC_SIZER_X_GAP, DCPOMATIC_SIZER_Y_GAP);
-       sizer->Add (_video_grid, 0, wxALL, 8);
-       panel->SetSizer (sizer);
+       auto panel = new wxPanel(_notebook);
+       auto sizer = new wxBoxSizer(wxVERTICAL);
+       _video_grid = new wxGridBagSizer(DCPOMATIC_SIZER_X_GAP, DCPOMATIC_SIZER_Y_GAP);
+       sizer->Add(_video_grid, 0, wxALL, 8);
+       panel->SetSizer(sizer);
 
-       _container_label = create_label (panel, _("Container"), true);
+       _container_label = create_label(panel, _("Container"), true);
        _container = new Choice(panel);
        _container_size = new StaticText(panel, {});
 
-       _resolution_label = create_label (panel, _("Resolution"), true);
+       _resolution_label = create_label(panel, _("Resolution"), true);
        _resolution = new Choice(panel);
 
-       _frame_rate_label = create_label (panel, _("Frame Rate"), true);
+       _frame_rate_label = create_label(panel, _("Frame Rate"), true);
        _frame_rate_choice = new Choice(panel);
-       _frame_rate_spin = new SpinCtrl (panel, DCPOMATIC_SPIN_CTRL_WIDTH);
-       _best_frame_rate = new Button (panel, _("Use best"));
+       _frame_rate_spin = new SpinCtrl(panel, DCPOMATIC_SPIN_CTRL_WIDTH);
+       _best_frame_rate = new Button(panel, _("Use best"));
 
-       _three_d = new CheckBox (panel, _("3D"));
+       _three_d = new CheckBox(panel, _("3D"));
 
        _video_bit_rate_label = create_label(panel, _("Video bit rate\nfor newly-encoded data"), true);
        _video_bit_rate = new SpinCtrl(panel, DCPOMATIC_SPIN_CTRL_WIDTH);
-       _mbits_label = create_label (panel, _("Mbit/s"), false);
+       _mbits_label = create_label(panel, _("Mbit/s"), false);
 
-       _reencode_j2k = new CheckBox (panel, _("Re-encode JPEG2000 data from input"));
+       _reencode_j2k = new CheckBox(panel, _("Re-encode JPEG2000 data from input"));
 
-       _container->Bind         (wxEVT_CHOICE,   boost::bind(&DCPPanel::container_changed, this));
-       _frame_rate_choice->Bind (wxEVT_CHOICE,   boost::bind(&DCPPanel::frame_rate_choice_changed, this));
-       _frame_rate_spin->Bind   (wxEVT_SPINCTRL, boost::bind(&DCPPanel::frame_rate_spin_changed, this));
-       _best_frame_rate->Bind   (wxEVT_BUTTON,   boost::bind(&DCPPanel::best_frame_rate_clicked, this));
-       _video_bit_rate->Bind    (wxEVT_SPINCTRL, boost::bind(&DCPPanel::video_bit_rate_changed, this));
+       _container->Bind        (wxEVT_CHOICE,    boost::bind(&DCPPanel::container_changed, this));
+       _frame_rate_choice->Bind(wxEVT_CHOICE,    boost::bind(&DCPPanel::frame_rate_choice_changed, this));
+       _frame_rate_spin->Bind  (wxEVT_SPINCTRL, boost::bind(&DCPPanel::frame_rate_spin_changed, this));
+       _best_frame_rate->Bind  (wxEVT_BUTTON,    boost::bind(&DCPPanel::best_frame_rate_clicked, this));
+       _video_bit_rate->Bind   (wxEVT_SPINCTRL, boost::bind(&DCPPanel::video_bit_rate_changed, this));
        /* Also listen to wxEVT_TEXT so that typing numbers directly in is always noticed */
-       _video_bit_rate->Bind    (wxEVT_TEXT,     boost::bind(&DCPPanel::video_bit_rate_changed, this));
-       _resolution->Bind        (wxEVT_CHOICE,   boost::bind(&DCPPanel::resolution_changed, this));
+       _video_bit_rate->Bind   (wxEVT_TEXT,     boost::bind(&DCPPanel::video_bit_rate_changed, this));
+       _resolution->Bind       (wxEVT_CHOICE,   boost::bind(&DCPPanel::resolution_changed, this));
        _three_d->bind(&DCPPanel::three_d_changed, this);
        _reencode_j2k->bind(&DCPPanel::reencode_j2k_changed, this);
 
@@ -837,12 +837,12 @@ DCPPanel::make_video_panel ()
 
        VideoEncoding const encoding = _film ? _film->video_encoding() : VideoEncoding::JPEG2000;
        _video_bit_rate->SetRange(1, Config::instance()->maximum_video_bit_rate(encoding) / 1000000);
-       _frame_rate_spin->SetRange (1, 480);
+       _frame_rate_spin->SetRange(1, 480);
 
        _resolution->add_entry(_("2K"));
        _resolution->add_entry(_("4K"));
 
-       add_video_panel_to_grid ();
+       add_video_panel_to_grid();
        setup_frame_rate_widget();
 
        return panel;
@@ -850,48 +850,48 @@ DCPPanel::make_video_panel ()
 
 
 void
-DCPPanel::add_video_panel_to_grid ()
+DCPPanel::add_video_panel_to_grid()
 {
        int r = 0;
 
-       add_label_to_sizer (_video_grid, _container_label, true, wxGBPosition (r, 0));
+       add_label_to_sizer(_video_grid, _container_label, true, wxGBPosition(r, 0));
        {
-               auto s = new wxBoxSizer (wxHORIZONTAL);
-               s->Add (_container, 1, wxEXPAND | wxRIGHT, DCPOMATIC_SIZER_X_GAP);
-               s->Add (_container_size, 1, wxLEFT | wxALIGN_CENTER_VERTICAL);
-               _video_grid->Add (s, wxGBPosition(r, 1));
+               auto s = new wxBoxSizer(wxHORIZONTAL);
+               s->Add(_container, 1, wxEXPAND | wxRIGHT, DCPOMATIC_SIZER_X_GAP);
+               s->Add(_container_size, 1, wxLEFT | wxALIGN_CENTER_VERTICAL);
+               _video_grid->Add(s, wxGBPosition(r, 1));
                ++r;
        }
 
-       add_label_to_sizer (_video_grid, _resolution_label, true, wxGBPosition (r, 0));
-       _video_grid->Add (_resolution, wxGBPosition (r, 1));
+       add_label_to_sizer(_video_grid, _resolution_label, true, wxGBPosition(r, 0));
+       _video_grid->Add(_resolution, wxGBPosition(r, 1));
        ++r;
 
-       add_label_to_sizer (_video_grid, _frame_rate_label, true, wxGBPosition (r, 0));
+       add_label_to_sizer(_video_grid, _frame_rate_label, true, wxGBPosition(r, 0));
        {
-               _frame_rate_sizer = new wxBoxSizer (wxHORIZONTAL);
-               _frame_rate_sizer->Add (_frame_rate_choice, 1, wxALIGN_CENTER_VERTICAL);
-               _frame_rate_sizer->Add (_frame_rate_spin, 1, wxALIGN_CENTER_VERTICAL);
-               _frame_rate_sizer->Add (_best_frame_rate, 1, wxLEFT | wxALIGN_CENTER_VERTICAL, DCPOMATIC_SIZER_X_GAP);
-               _video_grid->Add (_frame_rate_sizer, wxGBPosition (r, 1));
+               _frame_rate_sizer = new wxBoxSizer(wxHORIZONTAL);
+               _frame_rate_sizer->Add(_frame_rate_choice, 1, wxALIGN_CENTER_VERTICAL);
+               _frame_rate_sizer->Add(_frame_rate_spin, 1, wxALIGN_CENTER_VERTICAL);
+               _frame_rate_sizer->Add(_best_frame_rate, 1, wxLEFT | wxALIGN_CENTER_VERTICAL, DCPOMATIC_SIZER_X_GAP);
+               _video_grid->Add(_frame_rate_sizer, wxGBPosition(r, 1));
                ++r;
        }
 
-       _video_grid->Add (_three_d, wxGBPosition (r, 0), wxGBSpan (1, 2));
+       _video_grid->Add(_three_d, wxGBPosition(r, 0), wxGBSpan(1, 2));
        ++r;
 
-       add_label_to_sizer(_video_grid, _video_bit_rate_label, true, wxGBPosition (r, 0));
-       auto s = new wxBoxSizer (wxHORIZONTAL);
+       add_label_to_sizer(_video_grid, _video_bit_rate_label, true, wxGBPosition(r, 0));
+       auto s = new wxBoxSizer(wxHORIZONTAL);
        s->Add(_video_bit_rate, 0, wxALIGN_CENTER_VERTICAL);
-       add_label_to_sizer (s, _mbits_label, false, 0, wxLEFT | wxALIGN_CENTER_VERTICAL);
-       _video_grid->Add (s, wxGBPosition(r, 1), wxDefaultSpan);
+       add_label_to_sizer(s, _mbits_label, false, 0, wxLEFT | wxALIGN_CENTER_VERTICAL);
+       _video_grid->Add(s, wxGBPosition(r, 1), wxDefaultSpan);
        ++r;
-       _video_grid->Add (_reencode_j2k, wxGBPosition(r, 0), wxGBSpan(1, 2));
+       _video_grid->Add(_reencode_j2k, wxGBPosition(r, 0), wxGBSpan(1, 2));
 }
 
 
 int
-DCPPanel::minimum_allowed_audio_channels () const
+DCPPanel::minimum_allowed_audio_channels() const
 {
        int min = 2;
        if (_film) {
@@ -912,76 +912,76 @@ DCPPanel::minimum_allowed_audio_channels () const
 
 
 wxPanel *
-DCPPanel::make_audio_panel ()
+DCPPanel::make_audio_panel()
 {
-       auto panel = new wxPanel (_notebook);
-       _audio_panel_sizer = new wxBoxSizer (wxVERTICAL);
-       _audio_grid = new wxGridBagSizer (DCPOMATIC_SIZER_X_GAP, DCPOMATIC_SIZER_Y_GAP);
-       _audio_panel_sizer->Add (_audio_grid, 0, wxALL, 8);
-       panel->SetSizer (_audio_panel_sizer);
+       auto panel = new wxPanel(_notebook);
+       _audio_panel_sizer = new wxBoxSizer(wxVERTICAL);
+       _audio_grid = new wxGridBagSizer(DCPOMATIC_SIZER_X_GAP, DCPOMATIC_SIZER_Y_GAP);
+       _audio_panel_sizer->Add(_audio_grid, 0, wxALL, 8);
+       panel->SetSizer(_audio_panel_sizer);
 
-       _channels_label = create_label (panel, _("Channels"), true);
+       _channels_label = create_label(panel, _("Channels"), true);
        _audio_channels = new Choice(panel);
-       setup_audio_channels_choice (_audio_channels, minimum_allowed_audio_channels ());
+       setup_audio_channels_choice(_audio_channels, minimum_allowed_audio_channels());
 
        if (Config::instance()->allow_96khz_audio()) {
-               _audio_sample_rate_label = create_label (panel, _("Sample rate"), true);
-               _audio_sample_rate = new wxChoice (panel, wxID_ANY);
+               _audio_sample_rate_label = create_label(panel, _("Sample rate"), true);
+               _audio_sample_rate = new wxChoice(panel, wxID_ANY);
        }
 
-       _processor_label = create_label (panel, _("Processor"), true);
+       _processor_label = create_label(panel, _("Processor"), true);
        _audio_processor = new Choice(panel);
-       add_audio_processors ();
+       add_audio_processors();
 
        _enable_audio_language = new CheckBox(panel, _("Language"));
        _audio_language = new wxStaticText(panel, wxID_ANY, {});
        _edit_audio_language = new Button(panel, _("Edit..."));
 
-       _show_audio = new Button (panel, _("Show graph of audio levels..."));
+       _show_audio = new Button(panel, _("Show graph of audio levels..."));
 
-       _audio_channels->Bind (wxEVT_CHOICE, boost::bind (&DCPPanel::audio_channels_changed, this));
+       _audio_channels->Bind(wxEVT_CHOICE, boost::bind(&DCPPanel::audio_channels_changed, this));
        if (_audio_sample_rate) {
-               _audio_sample_rate->Bind (wxEVT_CHOICE, boost::bind(&DCPPanel::audio_sample_rate_changed, this));
+               _audio_sample_rate->Bind(wxEVT_CHOICE, boost::bind(&DCPPanel::audio_sample_rate_changed, this));
        }
-       _audio_processor->Bind (wxEVT_CHOICE, boost::bind (&DCPPanel::audio_processor_changed, this));
+       _audio_processor->Bind(wxEVT_CHOICE, boost::bind(&DCPPanel::audio_processor_changed, this));
 
        _enable_audio_language->bind(&DCPPanel::enable_audio_language_toggled, this);
        _edit_audio_language->Bind(wxEVT_BUTTON, boost::bind(&DCPPanel::edit_audio_language_clicked, this));
 
-       _show_audio->Bind (wxEVT_BUTTON, boost::bind (&DCPPanel::show_audio_clicked, this));
+       _show_audio->Bind(wxEVT_BUTTON, boost::bind(&DCPPanel::show_audio_clicked, this));
 
        if (_audio_sample_rate) {
-               _audio_sample_rate->Append (_("48kHz"));
-               _audio_sample_rate->Append (_("96kHz"));
+               _audio_sample_rate->Append(_("48kHz"));
+               _audio_sample_rate->Append(_("96kHz"));
        }
 
-       add_audio_panel_to_grid ();
+       add_audio_panel_to_grid();
 
        return panel;
 }
 
 
 void
-DCPPanel::add_audio_panel_to_grid ()
+DCPPanel::add_audio_panel_to_grid()
 {
        int r = 0;
 
-       add_label_to_sizer (_audio_grid, _channels_label, true, wxGBPosition (r, 0));
-       _audio_grid->Add (_audio_channels, wxGBPosition (r, 1));
+       add_label_to_sizer(_audio_grid, _channels_label, true, wxGBPosition(r, 0));
+       _audio_grid->Add(_audio_channels, wxGBPosition(r, 1));
        ++r;
 
        if (_audio_sample_rate_label && _audio_sample_rate) {
-               add_label_to_sizer (_audio_grid, _audio_sample_rate_label, true, wxGBPosition(r, 0));
-               _audio_grid->Add (_audio_sample_rate, wxGBPosition(r, 1));
+               add_label_to_sizer(_audio_grid, _audio_sample_rate_label, true, wxGBPosition(r, 0));
+               _audio_grid->Add(_audio_sample_rate, wxGBPosition(r, 1));
                ++r;
        }
 
-       add_label_to_sizer (_audio_grid, _processor_label, true, wxGBPosition (r, 0));
-       _audio_grid->Add (_audio_processor, wxGBPosition (r, 1));
+       add_label_to_sizer(_audio_grid, _processor_label, true, wxGBPosition(r, 0));
+       _audio_grid->Add(_audio_processor, wxGBPosition(r, 1));
        ++r;
 
        {
-               auto s = new wxBoxSizer (wxHORIZONTAL);
+               auto s = new wxBoxSizer(wxHORIZONTAL);
                s->Add(_enable_audio_language, 0, wxALIGN_CENTER_VERTICAL | wxRIGHT, DCPOMATIC_SIZER_GAP);
                s->Add(_audio_language, 1, wxALIGN_CENTER_VERTICAL | wxBOTTOM, DCPOMATIC_CHECKBOX_BOTTOM_PAD);
                s->Add(DCPOMATIC_SIZER_X_GAP, 0);
@@ -990,13 +990,13 @@ DCPPanel::add_audio_panel_to_grid ()
        }
        ++r;
 
-       _audio_grid->Add (_show_audio, wxGBPosition (r, 0), wxGBSpan (1, 2));
+       _audio_grid->Add(_show_audio, wxGBPosition(r, 0), wxGBSpan(1, 2));
        ++r;
 }
 
 
 void
-DCPPanel::copy_isdcf_name_button_clicked ()
+DCPPanel::copy_isdcf_name_button_clicked()
 {
        auto name = _film->name();
        if (name.length() > 20 && std::count(name.begin(), name.end(), '_') > 6) {
@@ -1005,25 +1005,25 @@ DCPPanel::copy_isdcf_name_button_clicked ()
                 */
                _film->set_name(name.substr(0, name.find("_")));
        }
-       _film->set_name (_film->isdcf_name (true));
-       _film->set_use_isdcf_name (false);
+       _film->set_name(_film->isdcf_name(true));
+       _film->set_use_isdcf_name(false);
 }
 
 
 void
-DCPPanel::audio_processor_changed ()
+DCPPanel::audio_processor_changed()
 {
        if (!_film || !_audio_processor->get()) {
                return;
        }
 
        auto const s = string_client_data(_audio_processor->GetClientObject(*_audio_processor->get()));
-       _film->set_audio_processor (AudioProcessor::from_id (s));
+       _film->set_audio_processor(AudioProcessor::from_id(s));
 }
 
 
 void
-DCPPanel::show_audio_clicked ()
+DCPPanel::show_audio_clicked()
 {
        if (!_film) {
                return;
@@ -1035,7 +1035,7 @@ DCPPanel::show_audio_clicked ()
 
 
 void
-DCPPanel::add_audio_processors ()
+DCPPanel::add_audio_processors()
 {
        _audio_processor->add_entry(_("None"), string{"none"});
        for (auto ap: AudioProcessor::visible()) {
@@ -1046,22 +1046,22 @@ DCPPanel::add_audio_processors ()
 
 
 void
-DCPPanel::enable_audio_language_toggled ()
+DCPPanel::enable_audio_language_toggled()
 {
-       setup_sensitivity ();
+       setup_sensitivity();
        if (_enable_audio_language->GetValue()) {
-               auto al = wx_to_std (_audio_language->GetLabel());
-               _film->set_audio_language (al.empty() ? dcp::LanguageTag("en-US") : dcp::LanguageTag(al));
+               auto al = wx_to_std(_audio_language->GetLabel());
+               _film->set_audio_language(al.empty() ? dcp::LanguageTag("en-US") : dcp::LanguageTag(al));
        } else {
-               _film->set_audio_language (boost::none);
+               _film->set_audio_language(boost::none);
        }
 }
 
 
 void
-DCPPanel::edit_audio_language_clicked ()
+DCPPanel::edit_audio_language_clicked()
 {
-       DCPOMATIC_ASSERT (_film->audio_language());
+       DCPOMATIC_ASSERT(_film->audio_language());
        LanguageTagDialog dialog(_panel, *_film->audio_language());
        if (dialog.ShowModal() == wxID_OK) {
               _film->set_audio_language(dialog.get());
@@ -1070,10 +1070,10 @@ DCPPanel::edit_audio_language_clicked ()
 
 
 void
-DCPPanel::audio_sample_rate_changed ()
+DCPPanel::audio_sample_rate_changed()
 {
        if (_audio_sample_rate) {
-               _film->set_audio_frame_rate (_audio_sample_rate->GetSelection() == 0 ? 48000 : 96000);
+               _film->set_audio_frame_rate(_audio_sample_rate->GetSelection() == 0 ? 48000 : 96000);
        }
 }
 
index 4988922a8b8dc10061124de2ae2a7146f6a3b585..2196e8927abf1768bf177835e1e822a64d32f76a 100644 (file)
@@ -52,64 +52,64 @@ class DCPPanel
 public:
        DCPPanel(wxNotebook *, std::shared_ptr<Film>, FilmViewer& viewer);
 
-       DCPPanel (DCPPanel const&) = delete;
-       DCPPanel& operator= (DCPPanel const&) = delete;
+       DCPPanel(DCPPanel const&) = delete;
+       DCPPanel& operator=(DCPPanel const&) = delete;
 
-       void set_film (std::shared_ptr<Film>);
-       void set_general_sensitivity (bool);
+       void set_film(std::shared_ptr<Film>);
+       void set_general_sensitivity(bool);
 
        void film_changed(FilmProperty);
-       void film_content_changed (int);
+       void film_content_changed(int);
 
-       wxPanel* panel () const {
+       wxPanel* panel() const {
                return _panel;
        }
 
 private:
-       void name_changed ();
-       void use_isdcf_name_toggled ();
-       void copy_isdcf_name_button_clicked ();
-       void container_changed ();
-       void dcp_content_type_changed ();
+       void name_changed();
+       void use_isdcf_name_toggled();
+       void copy_isdcf_name_button_clicked();
+       void container_changed();
+       void dcp_content_type_changed();
        void video_bit_rate_changed();
-       void frame_rate_choice_changed ();
-       void frame_rate_spin_changed ();
-       void best_frame_rate_clicked ();
-       void content_timeline_clicked ();
-       void audio_channels_changed ();
-       void resolution_changed ();
-       void three_d_changed ();
-       void standard_changed ();
-       void encrypted_toggled ();
-       void audio_processor_changed ();
-       void show_audio_clicked ();
-       void markers_clicked ();
-       void metadata_clicked ();
+       void frame_rate_choice_changed();
+       void frame_rate_spin_changed();
+       void best_frame_rate_clicked();
+       void content_timeline_clicked();
+       void audio_channels_changed();
+       void resolution_changed();
+       void three_d_changed();
+       void standard_changed();
+       void encrypted_toggled();
+       void audio_processor_changed();
+       void show_audio_clicked();
+       void markers_clicked();
+       void metadata_clicked();
        void reels_clicked();
-       void reencode_j2k_changed ();
-       void enable_audio_language_toggled ();
-       void edit_audio_language_clicked ();
-       void audio_sample_rate_changed ();
-
-       void setup_frame_rate_widget ();
-       void setup_container ();
-       void setup_dcp_name ();
-       void add_to_grid ();
-       void add_video_panel_to_grid ();
-       void add_audio_panel_to_grid ();
-       void add_audio_processors ();
+       void reencode_j2k_changed();
+       void enable_audio_language_toggled();
+       void edit_audio_language_clicked();
+       void audio_sample_rate_changed();
+
+       void setup_frame_rate_widget();
+       void setup_container();
+       void setup_dcp_name();
+       void add_to_grid();
+       void add_video_panel_to_grid();
+       void add_audio_panel_to_grid();
+       void add_audio_processors();
        void update_standards();
        void set_standard();
 
-       int minimum_allowed_audio_channels () const;
+       int minimum_allowed_audio_channels() const;
 
-       wxPanel* make_general_panel ();
-       wxPanel* make_video_panel ();
-       wxPanel* make_audio_panel ();
+       wxPanel* make_general_panel();
+       wxPanel* make_video_panel();
+       wxPanel* make_audio_panel();
 
-       void config_changed (Config::Property p);
+       void config_changed(Config::Property p);
 
-       void setup_sensitivity ();
+       void setup_sensitivity();
 
        wxPanel* _panel;
        wxNotebook* _notebook;