From abe21002bc84db630d860bc304261497cfd71f9b Mon Sep 17 00:00:00 2001 From: Carl Hetherington Date: Tue, 9 Feb 2021 13:58:37 +0100 Subject: [PATCH] Move some updates from content_change to player_change. So far as I can see there's no need for them to respond to content_change, and it's neater to do it all in one place. --- src/wx/film_viewer.cc | 18 ++++-------------- src/wx/film_viewer.h | 1 - 2 files changed, 4 insertions(+), 15 deletions(-) diff --git a/src/wx/film_viewer.cc b/src/wx/film_viewer.cc index bfacbdfe1..1993f0011 100644 --- a/src/wx/film_viewer.cc +++ b/src/wx/film_viewer.cc @@ -180,7 +180,6 @@ FilmViewer::set_film (shared_ptr film) _player->set_play_referenced (); _film->Change.connect (boost::bind (&FilmViewer::film_change, this, _1, _2)); - _film->ContentChange.connect (boost::bind(&FilmViewer::content_change, this, _1, _3)); _film->LengthChange.connect (boost::bind(&FilmViewer::film_length_change, this)); _player->Change.connect (boost::bind (&FilmViewer::player_change, this, _1, _2, _3)); @@ -402,6 +401,10 @@ FilmViewer::player_change (ChangeType type, int property, bool frequent) if (!refreshed) { slow_refresh (); } + + if (property == TextContentProperty::USE || property == TextContentProperty::TYPE || property == TextContentProperty::DCP_TRACK) { + _closed_captions_dialog->update_tracks (_film); + } } void @@ -707,19 +710,6 @@ FilmViewer::gets () const } -void -FilmViewer::content_change (ChangeType type, int property) -{ - if (type != ChangeType::DONE) { - return; - } - - if (property == TextContentProperty::USE || property == TextContentProperty::TYPE || property == TextContentProperty::DCP_TRACK) { - _closed_captions_dialog->update_tracks (_film); - } -} - - void FilmViewer::image_changed (shared_ptr pv) { diff --git a/src/wx/film_viewer.h b/src/wx/film_viewer.h index 9cbedc7cb..125e4fd2d 100644 --- a/src/wx/film_viewer.h +++ b/src/wx/film_viewer.h @@ -156,7 +156,6 @@ private: void idle_handler (); void request_idle_display_next_frame (); void film_change (ChangeType, Film::Property); - void content_change (ChangeType, int property); void recreate_butler (); void config_changed (Config::Property); void film_length_change (); -- 2.30.2