X-Git-Url: https://git.carlh.net/gitweb/?a=blobdiff_plain;f=src%2Fwx%2Ffilm_viewer.cc;h=5ca147b747b932be49c035c51b6e9477bffac12e;hb=7de09ede7bc6e34bea6ab3e7982efb51ca399b7d;hp=3f4bc6514048143df7f464280b3926d6ec9d4b5c;hpb=253c72987b333a747eeaf25509eb1ec32f484467;p=dcpomatic.git

diff --git a/src/wx/film_viewer.cc b/src/wx/film_viewer.cc
index 3f4bc6514..5ca147b74 100644
--- a/src/wx/film_viewer.cc
+++ b/src/wx/film_viewer.cc
@@ -37,6 +37,9 @@
 #include "lib/log.h"
 #include "film_viewer.h"
 #include "wx_util.h"
+extern "C" {
+#include <libavutil/pixfmt.h>
+}
 #include <dcp/exceptions.h>
 #include <wx/tglbtn.h>
 #include <iostream>
@@ -54,6 +57,7 @@ using std::exception;
 using boost::shared_ptr;
 using boost::dynamic_pointer_cast;
 using boost::weak_ptr;
+using boost::optional;
 using dcp::Size;
 
 FilmViewer::FilmViewer (wxWindow* p)
@@ -66,6 +70,8 @@ FilmViewer::FilmViewer (wxWindow* p)
 	, _frame_number (new wxStaticText (this, wxID_ANY, wxT("")))
 	, _timecode (new wxStaticText (this, wxID_ANY, wxT("")))
 	, _play_button (new wxToggleButton (this, wxID_ANY, _("Play")))
+	, _coalesce_player_changes (false)
+	, _pending_player_change (false)
 	, _last_get_accurate (true)
 {
 #ifndef __WXOSX__
@@ -113,7 +119,7 @@ FilmViewer::FilmViewer (wxWindow* p)
 	set_film (shared_ptr<Film> ());
 
 	JobManager::instance()->ActiveJobsChanged.connect (
-		bind (&FilmViewer::active_jobs_changed, this, _1)
+		bind (&FilmViewer::active_jobs_changed, this, _2)
 		);
 
 	setup_sensitivity ();
@@ -150,6 +156,7 @@ FilmViewer::set_film (shared_ptr<Film> film)
 	*/
 	_player->set_always_burn_subtitles (true);
 	_player->set_ignore_audio ();
+	_player->set_play_referenced ();
 
 	_film_connection = _film->Changed.connect (boost::bind (&FilmViewer::film_changed, this, _1));
 
@@ -184,7 +191,11 @@ FilmViewer::get (DCPTime p, bool accurate)
 
 	if (!pvf.empty ()) {
 		try {
-			_frame = pvf.front()->image (PIX_FMT_RGB24, boost::bind (&Log::dcp_log, _film->log().get(), _1, _2));
+			/* XXX: this could now give us a 48-bit image, which is a bit wasteful,
+			   or a XYZ image, which the code below will currently rely on FFmpeg
+			   to colourspace-convert.
+			*/
+			_frame = pvf.front()->image (boost::bind (&Log::dcp_log, _film->log().get(), _1, _2));
 			ImageChanged (pvf.front ());
 
 			dcp::YUVToRGB yuv_to_rgb = dcp::YUV_TO_RGB_REC601;
@@ -192,7 +203,7 @@ FilmViewer::get (DCPTime p, bool accurate)
 				yuv_to_rgb = pvf.front()->colour_conversion().get().yuv_to_rgb();
 			}
 
-			_frame = _frame->scale (_frame->size(), yuv_to_rgb, PIX_FMT_RGB24, false);
+			_frame = _frame->scale (_frame->size(), yuv_to_rgb, AV_PIX_FMT_RGB24, false);
 			_position = pvf.front()->time ();
 			_inter_position = pvf.front()->inter_position ();
 			_inter_size = pvf.front()->inter_size ();
@@ -385,23 +396,12 @@ FilmViewer::update_position_label ()
 }
 
 void
-FilmViewer::active_jobs_changed (bool a)
+FilmViewer::active_jobs_changed (optional<string> j)
 {
-	if (a) {
-		list<shared_ptr<Job> > jobs = JobManager::instance()->get ();
-		list<shared_ptr<Job> >::iterator i = jobs.begin ();
-		while (i != jobs.end() && boost::dynamic_pointer_cast<ExamineContentJob> (*i) == 0) {
-			++i;
-		}
-
-		if (i == jobs.end() || (*i)->finished()) {
-			/* no examine content job running, so we're ok to use the viewer */
-			a = false;
-		}
-	}
-
-	_slider->Enable (!a);
-	_play_button->Enable (!a);
+	/* examine content is the only job which stops the viewer working */
+	bool const a = !j || *j != "examine_content";
+	_slider->Enable (a);
+	_play_button->Enable (a);
 }
 
 void
@@ -437,6 +437,11 @@ FilmViewer::player_changed (bool frequent)
 		return;
 	}
 
+	if (_coalesce_player_changes) {
+		_pending_player_change = true;
+		return;
+	}
+
 	calculate_sizes ();
 	refresh ();
 	update_position_label ();
@@ -471,3 +476,26 @@ FilmViewer::refresh ()
 {
 	get (_position, _last_get_accurate);
 }
+
+void
+FilmViewer::set_position (DCPTime p)
+{
+	_position = p;
+	get (_position, true);
+	update_position_label ();
+	update_position_slider ();
+}
+
+void
+FilmViewer::set_coalesce_player_changes (bool c)
+{
+	_coalesce_player_changes = c;
+
+	if (c) {
+		_pending_player_change = false;
+	} else {
+		if (_pending_player_change) {
+			player_changed (false);
+		}
+	}
+}