Make EmptyVideo work with stereo a little better.

3 files changed, 53 insertions, 14 deletions

diff --git a/src/lib/empty_video.cc b/src/lib/empty_video.cc
index c578559e0..b99f911f2 100644
--- a/src/lib/empty_video.cc
+++ b/src/lib/empty_video.cc
@@ -35,23 +35,54 @@ using std::cout;
 using std::dynamic_pointer_cast;
 using std::function;
 using std::list;
+using std::make_pair;
+using std::pair;
 using std::shared_ptr;
 using namespace dcpomatic;
 
 
 EmptyVideo::EmptyVideo (shared_ptr<const Film> film, shared_ptr<const Playlist> playlist, DCPTime length)
 {
-	list<DCPTimePeriod> full;
+	list<DCPTimePeriod> full_left;
+	list<DCPTimePeriod> full_right;
+	list<DCPTimePeriod> full_both;
 	for (auto i: playlist->content()) {
 		if (i->video && i->video->use() && i->can_be_played() && i->paths_valid()) {
-			full.push_back (DCPTimePeriod(i->position(), i->end(film)));
+			auto period = DCPTimePeriod(i->position(), i->end(film));
+			if (i->video->frame_type() == VideoFrameType::THREE_D_LEFT) {
+				full_left.push_back (period);
+			} else if (i->video->frame_type() == VideoFrameType::THREE_D_RIGHT) {
+				full_right.push_back (period);
+			} else {
+				full_both.push_back (period);
+			}
 		}
 	}
 
-	_periods = subtract (DCPTimePeriod(DCPTime(), length), coalesce(full));
+	auto const whole = DCPTimePeriod(DCPTime(), length);
+
+	auto empty_both = subtract(subtract(subtract({whole}, full_left), full_right), full_both);
+	auto empty_left = subtract(subtract(subtract({whole}, full_left), full_both), empty_both);
+	auto empty_right = subtract(subtract(subtract({whole}, full_right), full_both), empty_both);
+
+	for (auto left: empty_left) {
+		_periods.push_back (make_pair(left, Eyes::LEFT));
+	}
+
+	for (auto right: empty_right) {
+		_periods.push_back (make_pair(right, Eyes::RIGHT));
+	}
+
+	for (auto both: empty_both) {
+		_periods.push_back (make_pair(both, Eyes::BOTH));
+	}
+
+	_periods.sort([](std::pair<DCPTimePeriod, Eyes> const& a, std::pair<DCPTimePeriod, Eyes> const& b) {
+		return a.first < b.first;
+	});
 
 	if (!_periods.empty()) {
-		_position = _periods.front().from;
+		_position = _periods.front().first.from;
 	}
 }
 
@@ -62,26 +93,26 @@ EmptyVideo::set_position (DCPTime position)
 	_position = position;
 
 	for (auto i: _periods) {
-		if (i.contains(_position)) {
+		if (i.first.contains(_position)) {
 			return;
 		}
 	}
 
 	for (auto i: _periods) {
-		if (i.from > _position) {
-			_position = i.from;
+		if (i.first.from > _position) {
+			_position = i.first.from;
 			return;
 		}
 	}
 }
 
 
-DCPTimePeriod
+pair<DCPTimePeriod, Eyes>
 EmptyVideo::period_at_position () const
 {
 	for (auto i: _periods) {
-		if (i.contains(_position)) {
-			return DCPTimePeriod (_position, i.to);
+		if (i.first.contains(_position)) {
+			return make_pair(DCPTimePeriod(_position, i.first.to), i.second);
 		}
 	}
 
@@ -94,7 +125,7 @@ EmptyVideo::done () const
 {
 	DCPTime latest;
 	for (auto i: _periods) {
-		latest = max (latest, i.to);
+		latest = max (latest, i.first.to);
 	}
 
 	return _position >= latest;
diff --git a/src/lib/empty_video.h b/src/lib/empty_video.h
index cf315bc44..5659ba504 100644
--- a/src/lib/empty_video.h
+++ b/src/lib/empty_video.h
@@ -27,12 +27,15 @@
 #include "dcpomatic_time.h"
 #include "playlist.h"
 #include <list>
+#include <utility>
 
 
 struct empty_video_test1;
 struct empty_video_test2;
 struct empty_video_test3;
 struct empty_video_test_with_overlapping_content;
+struct empty_video_test_with_three_d1;
+struct empty_video_test_with_three_d2;
 struct player_subframe_test;
 
 
@@ -46,7 +49,7 @@ public:
 		return _position;
 	}
 
-	dcpomatic::DCPTimePeriod period_at_position () const;
+	std::pair<dcpomatic::DCPTimePeriod, Eyes> period_at_position () const;
 
 	bool done () const;
 
@@ -57,9 +60,11 @@ private:
 	friend struct ::empty_video_test2;
 	friend struct ::empty_video_test3;
 	friend struct ::empty_video_test_with_overlapping_content;
+	friend struct empty_video_test_with_three_d1;
+	friend struct empty_video_test_with_three_d2;
 	friend struct ::player_subframe_test;
 
-	std::list<dcpomatic::DCPTimePeriod> _periods;
+	std::list<std::pair<dcpomatic::DCPTimePeriod, Eyes>> _periods;
 	dcpomatic::DCPTime _position;
 };
 
diff --git a/src/lib/player.cc b/src/lib/player.cc
index 66b731645..392c929a3 100644
--- a/src/lib/player.cc
+++ b/src/lib/player.cc
@@ -710,10 +710,13 @@ Player::pass ()
 		break;
 	}
 	case BLACK:
+	{
 		LOG_DEBUG_PLAYER ("Emit black for gap at %1", to_string(_black.position()));
-		emit_video (black_player_video_frame(Eyes::BOTH), _black.position());
+		auto period = _black.period_at_position();
+		emit_video (black_player_video_frame(period.second), _black.position());
 		_black.set_position (_black.position() + one_video_frame());
 		break;
+	}
 	case SILENT:
 	{
 		LOG_DEBUG_PLAYER ("Emit silence for gap at %1", to_string(_silent.position()));