X-Git-Url: https://main.carlh.net/gitweb/?p=dcpomatic.git;a=blobdiff_plain;f=src%2Flib%2Fvideo_decoder.cc;h=813320eb86f89ecd5518a315df121a9011500411;hp=b9ead52c8381680d2a8e97129a8d85c9d5b2f270;hb=254b3044d72de6b033d7c584f5abd2b9aa70aad5;hpb=3c634dda1e88fe04b0f468f39bd13ea41a80989b

diff --git a/src/lib/video_decoder.cc b/src/lib/video_decoder.cc
index b9ead52c8..813320eb8 100644
--- a/src/lib/video_decoder.cc
+++ b/src/lib/video_decoder.cc
@@ -1,5 +1,5 @@
 /*
-    Copyright (C) 2012-2016 Carl Hetherington <cth@carlh.net>
+    Copyright (C) 2012-2018 Carl Hetherington <cth@carlh.net>
 
     This file is part of DCP-o-matic.
 
@@ -36,8 +36,8 @@ using std::back_inserter;
 using boost::shared_ptr;
 using boost::optional;
 
-VideoDecoder::VideoDecoder (Decoder* parent, shared_ptr<const Content> c, shared_ptr<Log> log)
-	: DecoderPart (parent, log)
+VideoDecoder::VideoDecoder (Decoder* parent, shared_ptr<const Content> c)
+	: DecoderPart (parent)
 	, _content (c)
 {
 
@@ -53,17 +53,15 @@ VideoDecoder::VideoDecoder (Decoder* parent, shared_ptr<const Content> c, shared
  *  and so on.
  */
 void
-VideoDecoder::emit (shared_ptr<const ImageProxy> image, Frame frame)
+VideoDecoder::emit (shared_ptr<const Film> film, shared_ptr<const ImageProxy> image, Frame frame)
 {
 	if (ignore ()) {
 		return;
 	}
 
-	optional<bool> taken;
-
 	switch (_content->video->frame_type ()) {
 	case VIDEO_FRAME_TYPE_2D:
-		taken = Data (ContentVideo (image, frame, EYES_BOTH, PART_WHOLE));
+		Data (ContentVideo (image, frame, EYES_BOTH, PART_WHOLE));
 		break;
 	case VIDEO_FRAME_TYPE_3D:
 	{
@@ -71,35 +69,33 @@ VideoDecoder::emit (shared_ptr<const ImageProxy> image, Frame frame)
 		   frame this one is.
 		*/
 		bool const same = (_last_emitted && _last_emitted.get() == frame);
-		taken = Data (ContentVideo (image, frame, same ? EYES_RIGHT : EYES_LEFT, PART_WHOLE));
+		Data (ContentVideo (image, frame, same ? EYES_RIGHT : EYES_LEFT, PART_WHOLE));
 		_last_emitted = frame;
 		break;
 	}
 	case VIDEO_FRAME_TYPE_3D_ALTERNATE:
-		taken = Data (ContentVideo (image, frame / 2, (frame % 2) ? EYES_RIGHT : EYES_LEFT, PART_WHOLE));
+		Data (ContentVideo (image, frame / 2, (frame % 2) ? EYES_RIGHT : EYES_LEFT, PART_WHOLE));
 		frame /= 2;
 		break;
 	case VIDEO_FRAME_TYPE_3D_LEFT_RIGHT:
-		taken = Data (ContentVideo (image, frame, EYES_LEFT, PART_LEFT_HALF));
-		taken = Data (ContentVideo (image, frame, EYES_RIGHT, PART_RIGHT_HALF));
+		Data (ContentVideo (image, frame, EYES_LEFT, PART_LEFT_HALF));
+		Data (ContentVideo (image, frame, EYES_RIGHT, PART_RIGHT_HALF));
 		break;
 	case VIDEO_FRAME_TYPE_3D_TOP_BOTTOM:
-		taken = Data (ContentVideo (image, frame, EYES_LEFT, PART_TOP_HALF));
-		taken = Data (ContentVideo (image, frame, EYES_RIGHT, PART_BOTTOM_HALF));
+		Data (ContentVideo (image, frame, EYES_LEFT, PART_TOP_HALF));
+		Data (ContentVideo (image, frame, EYES_RIGHT, PART_BOTTOM_HALF));
 		break;
 	case VIDEO_FRAME_TYPE_3D_LEFT:
-		taken = Data (ContentVideo (image, frame, EYES_LEFT, PART_WHOLE));
+		Data (ContentVideo (image, frame, EYES_LEFT, PART_WHOLE));
 		break;
 	case VIDEO_FRAME_TYPE_3D_RIGHT:
-		taken = Data (ContentVideo (image, frame, EYES_RIGHT, PART_WHOLE));
+		Data (ContentVideo (image, frame, EYES_RIGHT, PART_WHOLE));
 		break;
 	default:
 		DCPOMATIC_ASSERT (false);
 	}
 
-	if (taken.get_value_or(false)) {
-		_position = ContentTime::from_frames (frame, _content->active_video_frame_rate ());
-	}
+	_position = ContentTime::from_frames (frame, _content->active_video_frame_rate(film));
 }
 
 void