X-Git-Url: https://main.carlh.net/gitweb/?a=blobdiff_plain;f=src%2Fwx%2Fclosed_captions_dialog.cc;h=7fcfc0808667a4d0d0271cda359044da98c5a088;hb=8fb0919c3c0f11f076eeda2a7d356f6284284ec6;hp=afcf5fa876f97b35d0cf66bf9765cf379e0afa13;hpb=579d18cb7770efe2da03afaf6a33faaf624119e3;p=dcpomatic.git diff --git a/src/wx/closed_captions_dialog.cc b/src/wx/closed_captions_dialog.cc index afcf5fa87..7fcfc0808 100644 --- a/src/wx/closed_captions_dialog.cc +++ b/src/wx/closed_captions_dialog.cc @@ -25,16 +25,19 @@ #include "lib/butler.h" #include "lib/text_content.h" #include "lib/compose.hpp" -#include +#include using std::list; using std::max; using std::cout; using std::pair; using std::make_pair; -using boost::shared_ptr; -using boost::weak_ptr; +using std::shared_ptr; +using std::weak_ptr; using boost::optional; +#if BOOST_VERSION >= 106100 +using namespace boost::placeholders; +#endif using namespace dcpomatic; ClosedCaptionsDialog::ClosedCaptionsDialog (wxWindow* parent, FilmViewer* viewer) @@ -55,12 +58,12 @@ ClosedCaptionsDialog::ClosedCaptionsDialog (wxWindow* parent, FilmViewer* viewer , _current_in_lines (false) , _timer (this) { - _lines.resize (CLOSED_CAPTION_LINES); + _lines.resize (MAX_CLOSED_CAPTION_LINES); wxBoxSizer* sizer = new wxBoxSizer (wxVERTICAL); wxBoxSizer* track_sizer = new wxBoxSizer (wxHORIZONTAL); - add_label_to_sizer (track_sizer, this, _("Track"), true); + add_label_to_sizer (track_sizer, this, _("Track"), true, 0, wxLEFT | wxRIGHT | wxALIGN_CENTRE_VERTICAL); track_sizer->Add (_track, 0, wxEXPAND | wxLEFT, DCPOMATIC_SIZER_X_GAP); sizer->Add (track_sizer, 0, wxALL, DCPOMATIC_SIZER_GAP); @@ -101,16 +104,16 @@ ClosedCaptionsDialog::paint () dc.SetTextForeground (*wxWHITE); /* Choose a font which fits vertically */ - int const line_height = max (8, dc.GetSize().GetHeight() / CLOSED_CAPTION_LINES); + int const line_height = max (8, dc.GetSize().GetHeight() / MAX_CLOSED_CAPTION_LINES); wxFont font (*wxNORMAL_FONT); font.SetPixelSize (wxSize (0, line_height * 0.8)); dc.SetFont (font); - for (int i = 0; i < CLOSED_CAPTION_LINES; ++i) { - wxString const good = _lines[i].Left (CLOSED_CAPTION_LENGTH); + for (int i = 0; i < MAX_CLOSED_CAPTION_LINES; ++i) { + wxString const good = _lines[i].Left (MAX_CLOSED_CAPTION_LENGTH); dc.DrawText (good, 8, line_height * i); - if (_lines[i].Length() > CLOSED_CAPTION_LENGTH) { - wxString const bad = _lines[i].Right (_lines[i].Length() - CLOSED_CAPTION_LENGTH); + if (_lines[i].Length() > MAX_CLOSED_CAPTION_LENGTH) { + wxString const bad = _lines[i].Right (_lines[i].Length() - MAX_CLOSED_CAPTION_LENGTH); wxSize size = dc.GetTextExtent (good); dc.SetTextForeground (*wxRED); dc.DrawText (bad, 8 + size.GetWidth(), line_height * i); @@ -131,11 +134,11 @@ private: float from_top (StringText const & c) const { switch (c.v_align()) { - case dcp::VALIGN_TOP: + case dcp::VAlign::TOP: return c.v_position(); - case dcp::VALIGN_CENTER: + case dcp::VAlign::CENTER: return c.v_position() + 0.5; - case dcp::VALIGN_BOTTOM: + case dcp::VAlign::BOTTOM: return 1.0 - c.v_position(); } DCPOMATIC_ASSERT (false); @@ -146,7 +149,7 @@ private: void ClosedCaptionsDialog::update () { - DCPTime const time = _viewer->time (); + auto const time = _viewer->time (); if (_current_in_lines && _current && _current->period.to > time) { /* Current one is fine */ @@ -155,7 +158,7 @@ ClosedCaptionsDialog::update () if (_current && _current->period.to < time) { /* Current one has finished; clear out */ - for (int j = 0; j < CLOSED_CAPTION_LINES; ++j) { + for (int j = 0; j < MAX_CLOSED_CAPTION_LINES; ++j) { _lines[j] = ""; } Refresh (); @@ -164,40 +167,41 @@ ClosedCaptionsDialog::update () if (!_current && !_tracks.empty()) { /* We have no current one: get another */ - shared_ptr butler = _butler.lock (); - DCPOMATIC_ASSERT (butler); + auto butler = _butler.lock (); DCPOMATIC_ASSERT (_track->GetSelection() >= 0); DCPOMATIC_ASSERT (_track->GetSelection() < int(_tracks.size())); - DCPTextTrack track = _tracks[_track->GetSelection()]; - while (true) { - optional d = butler->get_closed_caption (); - if (!d) { - break; - } - if (d->track == track) { - _current = d; - break; + auto track = _tracks[_track->GetSelection()]; + if (butler) { + while (true) { + optional d = butler->get_closed_caption (); + if (!d) { + break; + } + if (d->track == track) { + _current = d; + break; + } } - } - _current_in_lines = false; + _current_in_lines = false; + } } if (_current && _current->period.contains(time)) { /* We need to set this new one up */ - list to_show = _current->text.string; + auto to_show = _current->text.string; - for (int j = 0; j < CLOSED_CAPTION_LINES; ++j) { + for (int j = 0; j < MAX_CLOSED_CAPTION_LINES; ++j) { _lines[j] = ""; } to_show.sort (ClosedCaptionSorter()); - list::const_iterator j = to_show.begin(); + auto j = to_show.begin(); int k = 0; - while (j != to_show.end() && k < CLOSED_CAPTION_LINES) { - _lines[k] = j->text(); + while (j != to_show.end() && k < MAX_CLOSED_CAPTION_LINES) { + _lines[k] = std_to_wx (j->text()); ++j; ++k; } @@ -205,6 +209,12 @@ ClosedCaptionsDialog::update () Refresh (); _current_in_lines = true; } + + if (!_current && _tracks.empty()) { + for (int i = 0; i < MAX_CLOSED_CAPTION_LINES; ++i) { + _lines[i] = wxString(); + } + } } void @@ -215,14 +225,21 @@ ClosedCaptionsDialog::clear () Refresh (); } + +void +ClosedCaptionsDialog::set_butler (weak_ptr butler) +{ + _butler = butler; +} + void -ClosedCaptionsDialog::set_film_and_butler (shared_ptr film, weak_ptr butler) +ClosedCaptionsDialog::update_tracks (shared_ptr film) { _tracks.clear (); - BOOST_FOREACH (shared_ptr i, film->content()) { - BOOST_FOREACH (shared_ptr j, i->text) { - if (j->use() && j->type() == TEXT_CLOSED_CAPTION && j->dcp_track()) { + for (auto i: film->content()) { + for (auto j: i->text) { + if (j->use() && j->type() == TextType::CLOSED_CAPTION && j->dcp_track()) { if (find(_tracks.begin(), _tracks.end(), j->dcp_track()) == _tracks.end()) { _tracks.push_back (*j->dcp_track()); } @@ -231,13 +248,13 @@ ClosedCaptionsDialog::set_film_and_butler (shared_ptr film, weak_ptrClear (); - BOOST_FOREACH (DCPTextTrack const & i, _tracks) { - _track->Append (std_to_wx(String::compose("%1 (%2)", i.name, i.language))); + for (auto const& i: _tracks) { + _track->Append (std_to_wx(String::compose("%1 (%2)", i.name, i.language ? i.language->to_string() : wx_to_std(_("Unknown"))))); } if (_track->GetCount() > 0) { _track->SetSelection (0); } - _butler = butler; + track_selected (); }