X-Git-Url: https://git.carlh.net/gitweb/?a=blobdiff_plain;f=src%2Fwx%2Fplaylist_controls.cc;h=c098aa970dfd2413940e38bcde7651c35b75a6a5;hb=d0188ed3e1ea5536e729e93b87d54114368120ac;hp=da7976b8d7259be1223568b516e2b47102bc39ad;hpb=313319ba2d8544bc25524e02e634804a503b54f1;p=dcpomatic.git

diff --git a/src/wx/playlist_controls.cc b/src/wx/playlist_controls.cc
index da7976b8d..c098aa970 100644
--- a/src/wx/playlist_controls.cc
+++ b/src/wx/playlist_controls.cc
@@ -26,16 +26,20 @@
 #include "static_text.h"
 #include "wx_util.h"
 #include "lib/compose.hpp"
+#include "lib/constants.h"
 #include "lib/cross.h"
 #include "lib/dcp_content.h"
 #include "lib/ffmpeg_content.h"
 #include "lib/internet.h"
 #include "lib/player_video.h"
 #include "lib/scoped_temporary.h"
-#include <dcp/raw_convert.h>
 #include <dcp/exceptions.h>
+#include <dcp/raw_convert.h>
+#include <dcp/warnings.h>
+LIBDCP_DISABLE_WARNINGS
 #include <wx/listctrl.h>
 #include <wx/progdlg.h>
+LIBDCP_ENABLE_WARNINGS
 
 
 using std::cout;
@@ -48,7 +52,7 @@ using boost::optional;
 using namespace dcpomatic;
 
 
-PlaylistControls::PlaylistControls (wxWindow* parent, shared_ptr<FilmViewer> viewer)
+PlaylistControls::PlaylistControls(wxWindow* parent, FilmViewer& viewer)
 	: Controls (parent, viewer, false)
 	, _play_button (new Button(this, _("Play")))
 	, _pause_button (new Button(this, _("Pause")))
@@ -112,7 +116,7 @@ PlaylistControls::PlaylistControls (wxWindow* parent, shared_ptr<FilmViewer> vie
 	_previous_button->Bind (wxEVT_BUTTON, boost::bind(&PlaylistControls::previous_clicked,  this));
 	_spl_view->Bind        (wxEVT_LIST_ITEM_SELECTED,   boost::bind(&PlaylistControls::spl_selection_changed, this));
 	_spl_view->Bind        (wxEVT_LIST_ITEM_DESELECTED, boost::bind(&PlaylistControls::spl_selection_changed, this));
-	viewer->Finished.connect (boost::bind(&PlaylistControls::viewer_finished, this));
+	_viewer.Finished.connect(boost::bind(&PlaylistControls::viewer_finished, this));
 	_refresh_spl_view->Bind (wxEVT_BUTTON, boost::bind(&PlaylistControls::update_playlist_directory, this));
 	_refresh_content_view->Bind (wxEVT_BUTTON, boost::bind(&ContentView::update, _content_view));
 
@@ -145,32 +149,24 @@ PlaylistControls::deselect_playlist ()
 		_selected_playlist = boost::none;
 		_spl_view->SetItemState (selected, 0, wxLIST_STATE_SELECTED);
 	}
-	ResetFilm (shared_ptr<Film>(new Film(optional<boost::filesystem::path>())));
+	ResetFilm(std::make_shared<Film>(optional<boost::filesystem::path>()));
 }
 
 void
 PlaylistControls::play_clicked ()
 {
-	auto viewer = _viewer.lock ();
-	if (viewer) {
-		viewer->start ();
-	}
+	_viewer.start();
 }
 
 void
 PlaylistControls::setup_sensitivity ()
 {
-	auto viewer = _viewer.lock ();
-	if (!viewer) {
-		return;
-	}
-
 	Controls::setup_sensitivity ();
 	bool const active_job = _active_job && *_active_job != "examine_content";
 	bool const c = _film && !_film->content().empty() && !active_job;
-	_play_button->Enable (c && !viewer->playing());
-	_pause_button->Enable (viewer->playing());
-	_spl_view->Enable (!viewer->playing());
+	_play_button->Enable(c && !_viewer.playing());
+	_pause_button->Enable(_viewer.playing());
+	_spl_view->Enable(!_viewer.playing());
 	_next_button->Enable (can_do_next());
 	_previous_button->Enable (can_do_previous());
 }
@@ -178,22 +174,14 @@ PlaylistControls::setup_sensitivity ()
 void
 PlaylistControls::pause_clicked ()
 {
-	auto viewer = _viewer.lock ();
-	if (viewer) {
-		viewer->stop ();
-	}
+	_viewer.stop();
 }
 
 void
 PlaylistControls::stop_clicked ()
 {
-	auto viewer = _viewer.lock ();
-	if (!viewer) {
-		return;
-	}
-
-	viewer->stop ();
-	viewer->seek (DCPTime(), true);
+	_viewer.stop();
+	_viewer.seek(DCPTime(), true);
 	if (_selected_playlist) {
 		_selected_playlist_position = 0;
 		update_current_content ();
@@ -431,8 +419,7 @@ PlaylistControls::update_current_content ()
 void
 PlaylistControls::viewer_finished ()
 {
-	auto viewer = _viewer.lock ();
-	if (!_selected_playlist || !viewer) {
+	if (!_selected_playlist) {
 		return;
 	}
 
@@ -440,11 +427,11 @@ PlaylistControls::viewer_finished ()
 	if (_selected_playlist_position < int(_playlists[*_selected_playlist].get().size())) {
 		/* Next piece of content on the SPL */
 		update_current_content ();
-		viewer->start ();
+		_viewer.start();
 	} else {
 		/* Finished the whole SPL */
 		_selected_playlist_position = 0;
-		ResetFilm (shared_ptr<Film>(new Film(optional<boost::filesystem::path>())));
+		ResetFilm(std::make_shared<Film>(optional<boost::filesystem::path>()));
 		_play_button->Enable (true);
 		_pause_button->Enable (false);
 	}