From: Carl Hetherington Date: Sun, 21 Mar 2021 20:01:43 +0000 (+0100) Subject: C++11 tidying. X-Git-Tag: v2.15.137~15 X-Git-Url: https://git.carlh.net/gitweb/?p=dcpomatic.git;a=commitdiff_plain;h=6d53c07bb08e4437eaeec30e0469ad992b3d5372 C++11 tidying. --- diff --git a/src/wx/smpte_metadata_dialog.cc b/src/wx/smpte_metadata_dialog.cc index 64afcac5c..27ab94502 100644 --- a/src/wx/smpte_metadata_dialog.cc +++ b/src/wx/smpte_metadata_dialog.cc @@ -1,5 +1,5 @@ /* - Copyright (C) 2019-2020 Carl Hetherington + Copyright (C) 2019-2021 Carl Hetherington This file is part of DCP-o-matic. @@ -18,6 +18,7 @@ */ + #include "content_version_dialog.h" #include "editable_list.h" #include "language_tag_dialog.h" @@ -29,6 +30,7 @@ #include #include + using std::string; using std::vector; using boost::optional; @@ -68,10 +70,10 @@ SMPTEMetadataDialog::SMPTEMetadataDialog (wxWindow* parent, weak_ptr weak_ : wxDialog (parent, wxID_ANY, _("Metadata")) , WeakFilm (weak_film) { - wxBoxSizer* overall_sizer = new wxBoxSizer (wxVERTICAL); + auto overall_sizer = new wxBoxSizer (wxVERTICAL); SetSizer (overall_sizer); - wxFlexGridSizer* sizer = new wxFlexGridSizer (2, DCPOMATIC_SIZER_X_GAP, DCPOMATIC_SIZER_Y_GAP); + auto sizer = new wxFlexGridSizer (2, DCPOMATIC_SIZER_X_GAP, DCPOMATIC_SIZER_Y_GAP); sizer->AddGrowableCol (1, 1); add_label_to_sizer (sizer, this, _("Title language"), true, 0, wxLEFT | wxRIGHT | wxALIGN_CENTER_VERTICAL); @@ -122,7 +124,7 @@ SMPTEMetadataDialog::SMPTEMetadataDialog (wxWindow* parent, weak_ptr weak_ ); sizer->Add (_additional_subtitle_languages, 1, wxEXPAND); - Button* edit_release_territory = 0; + Button* edit_release_territory = nullptr; add_label_to_sizer (sizer, this, _("Release territory"), true, 0, wxLEFT | wxRIGHT | wxALIGN_CENTER_VERTICAL); { wxSizer* s = new wxBoxSizer (wxHORIZONTAL); @@ -155,7 +157,7 @@ SMPTEMetadataDialog::SMPTEMetadataDialog (wxWindow* parent, weak_ptr weak_ add_label_to_sizer (sizer, this, _("Luminance"), true, 0, wxLEFT | wxRIGHT | wxALIGN_CENTER_VERTICAL); { - wxSizer* s = new wxBoxSizer (wxHORIZONTAL); + auto s = new wxBoxSizer (wxHORIZONTAL); _luminance_value = new wxSpinCtrlDouble (this, wxID_ANY); _luminance_value->SetDigits (1); _luminance_value->SetIncrement (0.1); @@ -193,7 +195,7 @@ SMPTEMetadataDialog::SMPTEMetadataDialog (wxWindow* parent, weak_ptr weak_ #ifdef __WXOSX__ flags |= wxALIGN_RIGHT; #endif - wxStaticText* m = create_label (this, _("Content versions"), true); + auto m = create_label (this, _("Content versions"), true); sizer->Add (m, 0, flags, DCPOMATIC_SIZER_GAP); } @@ -212,7 +214,7 @@ SMPTEMetadataDialog::SMPTEMetadataDialog (wxWindow* parent, weak_ptr weak_ overall_sizer->Add (sizer, 1, wxEXPAND | wxALL, DCPOMATIC_DIALOG_BORDER); - wxSizer* buttons = CreateSeparatedButtonSizer (wxCLOSE); + auto buttons = CreateSeparatedButtonSizer (wxCLOSE); if (buttons) { overall_sizer->Add (buttons, wxSizerFlags().Expand().DoubleBorder()); } @@ -458,7 +460,7 @@ SMPTEMetadataDialog::setup_sensitivity () void SMPTEMetadataDialog::main_subtitle_language_changed (dcp::LanguageTag tag) { - vector existing = film()->subtitle_languages(); + auto existing = film()->subtitle_languages(); if (existing.empty()) { existing.push_back (tag); } else { @@ -472,7 +474,7 @@ SMPTEMetadataDialog::main_subtitle_language_changed (dcp::LanguageTag tag) vector SMPTEMetadataDialog::additional_subtitle_languages () { - vector all = film()->subtitle_languages(); + auto all = film()->subtitle_languages(); if (all.empty()) { return all; } @@ -484,7 +486,7 @@ SMPTEMetadataDialog::additional_subtitle_languages () void SMPTEMetadataDialog::set_additional_subtitle_languages (vector languages) { - vector all = film()->subtitle_languages(); + auto all = film()->subtitle_languages(); DCPOMATIC_ASSERT (!all.empty()); all.resize (1); copy (languages.begin(), languages.end(), back_inserter(all));