X-Git-Url: https://git.carlh.net/gitweb/?a=blobdiff_plain;f=src%2Flib%2Fplayer.cc;h=7e21ef937b82a98f5ff3198ac07c2ea0a469cbfb;hb=0253b4e45c71a1c2e2a8bffaf1c3cb84a0a3e41a;hp=035af81feba6215ee61a52eac3fa453f3fa205f8;hpb=b52491bd55097c343cd8514612d429a55d878be1;p=dcpomatic.git

diff --git a/src/lib/player.cc b/src/lib/player.cc
index 035af81fe..7e21ef937 100644
--- a/src/lib/player.cc
+++ b/src/lib/player.cc
@@ -47,7 +47,6 @@
 #include "content_subtitle.h"
 #include "dcp_decoder.h"
 #include "image_decoder.h"
-#include "resampler.h"
 #include "compose.hpp"
 #include <dcp/reel.h>
 #include <dcp/reel_sound_asset.h>
@@ -83,7 +82,7 @@ Player::Player (shared_ptr<const Film> film, shared_ptr<const Playlist> playlist
 	, _playlist (playlist)
 	, _have_valid_pieces (false)
 	, _ignore_video (false)
-	, _ignore_audio (false)
+	, _ignore_subtitle (false)
 	, _always_burn_subtitles (false)
 	, _fast (false)
 	, _play_referenced (false)
@@ -122,13 +121,16 @@ Player::setup_pieces ()
 			decoder->video->set_ignore ();
 		}
 
-		if (decoder->audio && _ignore_audio) {
-			decoder->audio->set_ignore ();
+		if (decoder->subtitle && _ignore_subtitle) {
+			decoder->subtitle->set_ignore ();
 		}
 
 		shared_ptr<DCPDecoder> dcp = dynamic_pointer_cast<DCPDecoder> (decoder);
 		if (dcp && _play_referenced) {
-			dcp->set_decode_referenced ();
+			if (_play_referenced) {
+				dcp->set_decode_referenced ();
+			}
+			dcp->set_forced_reduction (_dcp_decode_reduction);
 		}
 
 		shared_ptr<Piece> piece (new Piece (i, decoder, frc));
@@ -143,8 +145,9 @@ Player::setup_pieces ()
 		}
 
 		if (decoder->subtitle) {
-			decoder->subtitle->ImageData.connect (bind (&Player::image_subtitle, this, weak_ptr<Piece> (piece), _1));
-			decoder->subtitle->TextData.connect (bind (&Player::text_subtitle, this, weak_ptr<Piece> (piece), _1));
+			decoder->subtitle->ImageStart.connect (bind (&Player::image_subtitle_start, this, weak_ptr<Piece> (piece), _1));
+			decoder->subtitle->TextStart.connect (bind (&Player::text_subtitle_start, this, weak_ptr<Piece> (piece), _1));
+			decoder->subtitle->Stop.connect (bind (&Player::subtitle_stop, this, weak_ptr<Piece> (piece), _1));
 		}
 	}
 
@@ -156,20 +159,11 @@ Player::setup_pieces ()
 		}
 	}
 
-	if (!_play_referenced) {
-		BOOST_FOREACH (shared_ptr<Piece> i, _pieces) {
-			shared_ptr<DCPContent> dc = dynamic_pointer_cast<DCPContent> (i->content);
-			if (dc) {
-				if (dc->reference_video()) {
-					_no_video.push_back (DCPTimePeriod (dc->position(), dc->end()));
-				}
-				if (dc->reference_audio()) {
-					_no_audio.push_back (DCPTimePeriod (dc->position(), dc->end()));
-				}
-			}
-		}
-	}
+	_black = Empty (_film->content(), _film->length(), bind(&Content::video, _1));
+	_silent = Empty (_film->content(), _film->length(), bind(&Content::audio, _1));
 
+	_last_video_time = DCPTime ();
+	_last_audio_time = DCPTime ();
 	_have_valid_pieces = true;
 }
 
@@ -226,10 +220,16 @@ Player::playlist_content_changed (weak_ptr<Content> w, int property, bool freque
 void
 Player::set_video_container_size (dcp::Size s)
 {
+	if (s == _video_container_size) {
+		return;
+	}
+
 	_video_container_size = s;
 
 	_black_image.reset (new Image (AV_PIX_FMT_RGB24, _video_container_size, true));
 	_black_image->make_black ();
+
+	Changed (false);
 }
 
 void
@@ -362,8 +362,9 @@ DCPTime
 Player::resampled_audio_to_dcp (shared_ptr<const Piece> piece, Frame f) const
 {
 	/* See comment in dcp_to_content_video */
-	DCPTime const d = DCPTime::from_frames (f, _film->audio_frame_rate()) - DCPTime (piece->content->trim_start (), piece->frc);
-	return max (DCPTime (), d + piece->content->position ());
+	return DCPTime::from_frames (f, _film->audio_frame_rate())
+		- DCPTime (piece->content->trim_start(), piece->frc)
+		+ piece->content->position();
 }
 
 ContentTime
@@ -408,11 +409,10 @@ Player::set_ignore_video ()
 	_ignore_video = true;
 }
 
-/** Set this player never to produce any audio data */
 void
-Player::set_ignore_audio ()
+Player::set_ignore_subtitle ()
 {
-	_ignore_audio = true;
+	_ignore_subtitle = true;
 }
 
 /** Set whether or not this player should always burn text subtitles into the image,
@@ -505,80 +505,134 @@ Player::get_reel_assets ()
 	return a;
 }
 
-list<shared_ptr<Piece> >
-Player::overlaps (DCPTime from, DCPTime to, boost::function<bool (Content *)> valid)
+bool
+Player::pass ()
 {
 	if (!_have_valid_pieces) {
 		setup_pieces ();
 	}
 
-	list<shared_ptr<Piece> > overlaps;
-	BOOST_FOREACH (shared_ptr<Piece> i, _pieces) {
-		if (valid (i->content.get ()) && i->content->position() < to && i->content->end() > from) {
-			overlaps.push_back (i);
-		}
+	if (_playlist->length() == DCPTime()) {
+		/* Special case of an empty Film; just give one black frame */
+		emit_video (black_player_video_frame(), DCPTime());
+		return true;
 	}
 
-	return overlaps;
-}
-
-bool
-Player::pass ()
-{
-	if (!_have_valid_pieces) {
-		setup_pieces ();
-	}
+	/* Find the decoder or empty which is farthest behind where we are and make it emit some data */
 
-	shared_ptr<Piece> earliest;
-	DCPTime earliest_content;
+	shared_ptr<Piece> earliest_content;
+	optional<DCPTime> earliest_time;
 
 	BOOST_FOREACH (shared_ptr<Piece> i, _pieces) {
 		if (!i->done) {
-			DCPTime const t = i->content->position() + DCPTime (i->decoder->position(), i->frc);
-			if (!earliest || t < earliest_content) {
-				earliest_content = t;
-				earliest = i;
+			DCPTime const t = content_time_to_dcp (i, i->decoder->position());
+			/* Given two choices at the same time, pick the one with a subtitle so we see it before
+			   the video.
+			*/
+			if (!earliest_time || t < *earliest_time || (t == *earliest_time && i->decoder->subtitle)) {
+				earliest_time = t;
+				earliest_content = i;
 			}
 		}
 	}
 
-	if (!earliest) {
-		/* No more content; fill up with silent black */
-		DCPTimePeriod remaining_video (DCPTime(), _playlist->length());
-		if (_last_time) {
-			remaining_video.from = _last_time.get() + one_video_frame();
-		}
-		fill_video (remaining_video);
-		fill_audio (DCPTimePeriod (_last_audio_time, _playlist->length()));
-		return true;
+	bool done = false;
+
+	enum {
+		NONE,
+		CONTENT,
+		BLACK,
+		SILENT
+	} which = NONE;
+
+	if (earliest_content) {
+		which = CONTENT;
+	}
+
+	if (!_black.done() && (!earliest_time || _black.position() < *earliest_time)) {
+		earliest_time = _black.position ();
+		which = BLACK;
+	}
+
+	if (!_silent.done() && (!earliest_time || _silent.position() < *earliest_time)) {
+		earliest_time = _silent.position ();
+		which = SILENT;
 	}
 
-	earliest->done = earliest->decoder->pass ();
-	if (earliest->done && earliest->content->audio) {
-		/* Flush the Player audio system for this piece */
-		BOOST_FOREACH (AudioStreamPtr i, earliest->content->audio->streams()) {
-			audio_flush (earliest, i);
+	switch (which) {
+	case CONTENT:
+		earliest_content->done = earliest_content->decoder->pass ();
+		break;
+	case BLACK:
+		emit_video (black_player_video_frame(), _black.position());
+		_black.set_position (_black.position() + one_video_frame());
+		break;
+	case SILENT:
+	{
+		DCPTimePeriod period (_silent.period_at_position());
+		if (period.duration() > one_video_frame()) {
+			period.to = period.from + one_video_frame();
 		}
+		fill_audio (period);
+		_silent.set_position (period.to);
+		break;
+	}
+	case NONE:
+		done = true;
+		break;
 	}
 
 	/* Emit any audio that is ready */
 
-	DCPTime pull_from = _playlist->length ();
+	DCPTime pull_to = _film->length ();
 	for (map<AudioStreamPtr, StreamState>::const_iterator i = _stream_states.begin(); i != _stream_states.end(); ++i) {
-		if (!i->second.piece->done && i->second.last_push_end < pull_from) {
-			pull_from = i->second.last_push_end;
+		if (!i->second.piece->done && i->second.last_push_end < pull_to) {
+			pull_to = i->second.last_push_end;
 		}
 	}
 
-	list<pair<shared_ptr<AudioBuffers>, DCPTime> > audio = _audio_merger.pull (pull_from);
+	list<pair<shared_ptr<AudioBuffers>, DCPTime> > audio = _audio_merger.pull (pull_to);
 	for (list<pair<shared_ptr<AudioBuffers>, DCPTime> >::iterator i = audio.begin(); i != audio.end(); ++i) {
-		DCPOMATIC_ASSERT (i->second >= _last_audio_time);
-		fill_audio (DCPTimePeriod (_last_audio_time, i->second));
-		Audio (i->first, i->second);
-		_last_audio_time = i->second + DCPTime::from_frames(i->first->frames(), _film->audio_frame_rate());
+		if (_last_audio_time && i->second < *_last_audio_time) {
+			/* There has been an accurate seek and we have received some audio before the seek time;
+			   discard it.
+			*/
+			pair<shared_ptr<AudioBuffers>, DCPTime> cut = discard_audio (i->first, i->second, *_last_audio_time);
+			if (!cut.first) {
+				continue;
+			}
+			*i = cut;
+		}
+
+		emit_audio (i->first, i->second);
+	}
+
+	return done;
+}
+
+optional<PositionImage>
+Player::subtitles_for_frame (DCPTime time) const
+{
+	list<PositionImage> subtitles;
+
+	BOOST_FOREACH (PlayerSubtitles i, _active_subtitles.get_burnt (time, _always_burn_subtitles)) {
+
+		/* Image subtitles */
+		list<PositionImage> c = transform_image_subtitles (i.image);
+		copy (c.begin(), c.end(), back_inserter (subtitles));
+
+		/* Text subtitles (rendered to an image) */
+		if (!i.text.empty ()) {
+			list<PositionImage> s = render_subtitles (i.text, i.fonts, _video_container_size, time);
+			copy (s.begin(), s.end(), back_inserter (subtitles));
+		}
+	}
+
+	if (subtitles.empty ()) {
+		return optional<PositionImage> ();
 	}
 
-	return false;
+	return merge (subtitles);
 }
 
 void
@@ -594,49 +648,33 @@ Player::video (weak_ptr<Piece> wp, ContentVideo video)
 		return;
 	}
 
-	/* Time and period of the frame we will emit */
+	/* Time of the first frame we will emit */
 	DCPTime const time = content_video_to_dcp (piece, video.frame);
-	DCPTimePeriod const period (time, time + one_video_frame());
 
-	/* Discard if it's outside the content's period */
-	if (time < piece->content->position() || time >= piece->content->end()) {
+	/* Discard if it's outside the content's period or if it's before the last accurate seek */
+	if (
+		time < piece->content->position() ||
+		time >= piece->content->end() ||
+		(_last_video_time && time < *_last_video_time)) {
 		return;
 	}
 
-	/* Get any subtitles */
-
-	optional<PositionImage> subtitles;
+	/* Fill gaps that we discover now that we have some video which needs to be emitted */
 
-	for (list<pair<PlayerSubtitles, DCPTimePeriod> >::const_iterator i = _subtitles.begin(); i != _subtitles.end(); ++i) {
-
-		if (!i->second.overlap (period)) {
-			continue;
-		}
-
-		list<PositionImage> sub_images;
-
-		/* Image subtitles */
-		list<PositionImage> c = transform_image_subtitles (i->first.image);
-		copy (c.begin(), c.end(), back_inserter (sub_images));
-
-		/* Text subtitles (rendered to an image) */
-		if (!i->first.text.empty ()) {
-			list<PositionImage> s = render_subtitles (i->first.text, i->first.fonts, _video_container_size, time);
-			copy (s.begin (), s.end (), back_inserter (sub_images));
-		}
-
-		if (!sub_images.empty ()) {
-			subtitles = merge (sub_images);
+	if (_last_video_time) {
+		/* XXX: this may not work for 3D */
+		DCPTime fill_from = max (*_last_video_time, piece->content->position());
+		for (DCPTime j = fill_from; j < time; j += one_video_frame()) {
+			LastVideoMap::const_iterator k = _last_video.find (wp);
+			if (k != _last_video.end ()) {
+				emit_video (k->second, j);
+			} else {
+				emit_video (black_player_video_frame(), j);
+			}
 		}
 	}
 
-	/* Fill gaps */
-
-	if (_last_time) {
-		fill_video (DCPTimePeriod (_last_time.get() + one_video_frame(), time));
-	}
-
-	_last_video.reset (
+	_last_video[wp].reset (
 		new PlayerVideo (
 			video.image,
 			piece->content->video->crop (),
@@ -651,54 +689,55 @@ Player::video (weak_ptr<Piece> wp, ContentVideo video)
 			)
 		);
 
-	if (subtitles) {
-		_last_video->set_subtitle (subtitles.get ());
-	}
-
-	_last_time = time;
-
-	Video (_last_video, *_last_time);
-
-	/* Discard any subtitles we no longer need */
-
-	for (list<pair<PlayerSubtitles, DCPTimePeriod> >::iterator i = _subtitles.begin (); i != _subtitles.end(); ) {
-		list<pair<PlayerSubtitles, DCPTimePeriod> >::iterator tmp = i;
-		++tmp;
-
-		if (i->second.to < time) {
-			_subtitles.erase (i);
-		}
-
-		i = tmp;
+	DCPTime t = time;
+	for (int i = 0; i < frc.repeat; ++i) {
+		emit_video (_last_video[wp], t);
+		t += one_video_frame ();
 	}
 }
 
 void
-Player::audio_flush (shared_ptr<Piece> piece, AudioStreamPtr stream)
+Player::audio (weak_ptr<Piece> wp, AudioStreamPtr stream, ContentAudio content_audio)
 {
-	shared_ptr<AudioContent> content = piece->content->audio;
-	DCPOMATIC_ASSERT (content);
+	DCPOMATIC_ASSERT (content_audio.audio->frames() > 0);
 
-	shared_ptr<Resampler> r = resampler (content, stream, false);
-	if (!r) {
+	shared_ptr<Piece> piece = wp.lock ();
+	if (!piece) {
 		return;
 	}
 
-	pair<shared_ptr<const AudioBuffers>, Frame> ro = r->flush ();
-	ContentAudio content_audio;
-	content_audio.audio = ro.first;
-	content_audio.frame = ro.second;
+	shared_ptr<AudioContent> content = piece->content->audio;
+	DCPOMATIC_ASSERT (content);
 
 	/* Compute time in the DCP */
-	DCPTime time = resampled_audio_to_dcp (piece, content_audio.frame) + DCPTime::from_seconds (content->delay() / 1000.0);
+	DCPTime time = resampled_audio_to_dcp (piece, content_audio.frame);
+	/* And the end of this block in the DCP */
+	DCPTime end = time + DCPTime::from_frames(content_audio.audio->frames(), content->resampled_frame_rate());
 
-	audio_transform (content, stream, content_audio, time);
-}
+	/* Remove anything that comes before the start or after the end of the content */
+	if (time < piece->content->position()) {
+		pair<shared_ptr<AudioBuffers>, DCPTime> cut = discard_audio (content_audio.audio, time, piece->content->position());
+		if (!cut.first) {
+			/* This audio is entirely discarded */
+			return;
+		}
+		content_audio.audio = cut.first;
+		time = cut.second;
+	} else if (time > piece->content->end()) {
+		/* Discard it all */
+		return;
+	} else if (end > piece->content->end()) {
+		Frame const remaining_frames = DCPTime(piece->content->end() - time).frames_round(_film->audio_frame_rate());
+		if (remaining_frames == 0) {
+			return;
+		}
+		shared_ptr<AudioBuffers> cut (new AudioBuffers (content_audio.audio->channels(), remaining_frames));
+		cut->copy_from (content_audio.audio.get(), remaining_frames, 0, 0);
+		content_audio.audio = cut;
+	}
+
+	DCPOMATIC_ASSERT (content_audio.audio->frames() > 0);
 
-/** Do our common processing on some audio */
-void
-Player::audio_transform (shared_ptr<AudioContent> content, AudioStreamPtr stream, ContentAudio content_audio, DCPTime time)
-{
 	/* Gain */
 
 	if (content->gain() != 0) {
@@ -709,24 +748,7 @@ Player::audio_transform (shared_ptr<AudioContent> content, AudioStreamPtr stream
 
 	/* Remap */
 
-	shared_ptr<AudioBuffers> dcp_mapped (new AudioBuffers (_film->audio_channels(), content_audio.audio->frames()));
-	dcp_mapped->make_silent ();
-
-	AudioMapping map = stream->mapping ();
-	for (int i = 0; i < map.input_channels(); ++i) {
-		for (int j = 0; j < dcp_mapped->channels(); ++j) {
-			if (map.get (i, static_cast<dcp::Channel> (j)) > 0) {
-				dcp_mapped->accumulate_channel (
-					content_audio.audio.get(),
-					i,
-					static_cast<dcp::Channel> (j),
-					map.get (i, static_cast<dcp::Channel> (j))
-					);
-			}
-		}
-	}
-
-	content_audio.audio = dcp_mapped;
+	content_audio.audio = remap (content_audio.audio, _film->audio_channels(), stream->mapping());
 
 	/* Process */
 
@@ -742,49 +764,7 @@ Player::audio_transform (shared_ptr<AudioContent> content, AudioStreamPtr stream
 }
 
 void
-Player::audio (weak_ptr<Piece> wp, AudioStreamPtr stream, ContentAudio content_audio)
-{
-	shared_ptr<Piece> piece = wp.lock ();
-	if (!piece) {
-		return;
-	}
-
-	shared_ptr<AudioContent> content = piece->content->audio;
-	DCPOMATIC_ASSERT (content);
-
-	/* Resample */
-	if (stream->frame_rate() != content->resampled_frame_rate()) {
-		shared_ptr<Resampler> r = resampler (content, stream, true);
-		pair<shared_ptr<const AudioBuffers>, Frame> ro = r->run (content_audio.audio, content_audio.frame);
-		content_audio.audio = ro.first;
-		content_audio.frame = ro.second;
-	}
-
-	/* XXX: end-trimming used to be checked here */
-
-	/* Compute time in the DCP */
-	DCPTime time = resampled_audio_to_dcp (piece, content_audio.frame) + DCPTime::from_seconds (content->delay() / 1000.0);
-
-	/* Remove anything that comes before the start of the content */
-	if (time < piece->content->position()) {
-		DCPTime const discard_time = piece->content->position() - time;
-		Frame discard_frames = discard_time.frames_round(_film->audio_frame_rate());
-		Frame remaining_frames = content_audio.audio->frames() - discard_frames;
-		if (remaining_frames <= 0) {
-			/* This audio is entirely discarded */
-			return;
-		}
-		shared_ptr<AudioBuffers> cut (new AudioBuffers (content_audio.audio->channels(), remaining_frames));
-		cut->copy_from (content_audio.audio.get(), remaining_frames, discard_frames, 0);
-		content_audio.audio = cut;
-		time += discard_time;
-	}
-
-	audio_transform (content, stream, content_audio, time);
-}
-
-void
-Player::image_subtitle (weak_ptr<Piece> wp, ContentImageSubtitle subtitle)
+Player::image_subtitle_start (weak_ptr<Piece> wp, ContentImageSubtitle subtitle)
 {
 	shared_ptr<Piece> piece = wp.lock ();
 	if (!piece) {
@@ -805,17 +785,13 @@ Player::image_subtitle (weak_ptr<Piece> wp, ContentImageSubtitle subtitle)
 
 	PlayerSubtitles ps;
 	ps.image.push_back (subtitle.sub);
-	DCPTimePeriod period (content_time_to_dcp (piece, subtitle.period().from), content_time_to_dcp (piece, subtitle.period().to));
+	DCPTime from (content_time_to_dcp (piece, subtitle.from()));
 
-	if (piece->content->subtitle->use() && (piece->content->subtitle->burn() || _always_burn_subtitles)) {
-		_subtitles.push_back (make_pair (ps, period));
-	} else {
-		Subtitle (ps, period);
-	}
+	_active_subtitles.add_from (wp, ps, from);
 }
 
 void
-Player::text_subtitle (weak_ptr<Piece> wp, ContentTextSubtitle subtitle)
+Player::text_subtitle_start (weak_ptr<Piece> wp, ContentTextSubtitle subtitle)
 {
 	shared_ptr<Piece> piece = wp.lock ();
 	if (!piece) {
@@ -823,7 +799,7 @@ Player::text_subtitle (weak_ptr<Piece> wp, ContentTextSubtitle subtitle)
 	}
 
 	PlayerSubtitles ps;
-	DCPTimePeriod const period (content_time_to_dcp (piece, subtitle.period().from), content_time_to_dcp (piece, subtitle.period().to));
+	DCPTime const from (content_time_to_dcp (piece, subtitle.from()));
 
 	BOOST_FOREACH (dcp::SubtitleString s, subtitle.subs) {
 		s.set_h_position (s.h_position() + piece->content->subtitle->x_offset ());
@@ -845,97 +821,120 @@ Player::text_subtitle (weak_ptr<Piece> wp, ContentTextSubtitle subtitle)
 			s.set_aspect_adjust (xs / ys);
 		}
 
-		s.set_in (dcp::Time(period.from.seconds(), 1000));
-		s.set_out (dcp::Time(period.to.seconds(), 1000));
+		s.set_in (dcp::Time(from.seconds(), 1000));
 		ps.text.push_back (SubtitleString (s, piece->content->subtitle->outline_width()));
 		ps.add_fonts (piece->content->subtitle->fonts ());
 	}
 
-	if (piece->content->subtitle->use() && (piece->content->subtitle->burn() || _always_burn_subtitles)) {
-		_subtitles.push_back (make_pair (ps, period));
-	} else {
-		Subtitle (ps, period);
+	_active_subtitles.add_from (wp, ps, from);
+}
+
+void
+Player::subtitle_stop (weak_ptr<Piece> wp, ContentTime to)
+{
+	if (!_active_subtitles.have (wp)) {
+		return;
+	}
+
+	shared_ptr<Piece> piece = wp.lock ();
+	if (!piece) {
+		return;
+	}
+
+	DCPTime const dcp_to = content_time_to_dcp (piece, to);
+
+	pair<PlayerSubtitles, DCPTime> from = _active_subtitles.add_to (wp, dcp_to);
+
+	if (piece->content->subtitle->use() && !_always_burn_subtitles && !piece->content->subtitle->burn()) {
+		Subtitle (from.first, DCPTimePeriod (from.second, dcp_to));
 	}
 }
 
 void
 Player::seek (DCPTime time, bool accurate)
 {
+	if (!_have_valid_pieces) {
+		setup_pieces ();
+	}
+
 	if (_audio_processor) {
 		_audio_processor->flush ();
 	}
 
+	_audio_merger.clear ();
+	_active_subtitles.clear ();
+
 	BOOST_FOREACH (shared_ptr<Piece> i, _pieces) {
-		if (i->content->position() <= time && time < i->content->end()) {
+		if (time < i->content->position()) {
+			/* Before; seek to 0 */
+			i->decoder->seek (ContentTime(), accurate);
+			i->done = false;
+		} else if (i->content->position() <= time && time < i->content->end()) {
+			/* During; seek to position */
 			i->decoder->seek (dcp_to_content_time (i, time), accurate);
 			i->done = false;
+		} else {
+			/* After; this piece is done */
+			i->done = true;
 		}
 	}
 
 	if (accurate) {
-		_last_time = time - one_video_frame ();
+		_last_video_time = time;
+		_last_audio_time = time;
 	} else {
-		_last_time = optional<DCPTime> ();
+		_last_video_time = optional<DCPTime>();
+		_last_audio_time = optional<DCPTime>();
 	}
+
+	_black.set_position (time);
+	_silent.set_position (time);
+
+	_last_video.clear ();
 }
 
-shared_ptr<Resampler>
-Player::resampler (shared_ptr<const AudioContent> content, AudioStreamPtr stream, bool create)
+void
+Player::emit_video (shared_ptr<PlayerVideo> pv, DCPTime time)
 {
-	ResamplerMap::const_iterator i = _resamplers.find (make_pair (content, stream));
-	if (i != _resamplers.end ()) {
-		return i->second;
-	}
-
-	if (!create) {
-		return shared_ptr<Resampler> ();
+	optional<PositionImage> subtitles = subtitles_for_frame (time);
+	if (subtitles) {
+		pv->set_subtitle (subtitles.get ());
 	}
 
-	LOG_GENERAL (
-		"Creating new resampler from %1 to %2 with %3 channels",
-		stream->frame_rate(),
-		content->resampled_frame_rate(),
-		stream->channels()
-		);
+	Video (pv, time);
 
-	shared_ptr<Resampler> r (
-		new Resampler (stream->frame_rate(), content->resampled_frame_rate(), stream->channels())
-		);
-
-	_resamplers[make_pair(content, stream)] = r;
-	return r;
+	if (pv->eyes() == EYES_BOTH || pv->eyes() == EYES_RIGHT) {
+		_last_video_time = time + one_video_frame();
+		_active_subtitles.clear_before (time);
+	}
 }
 
 void
-Player::fill_video (DCPTimePeriod period)
+Player::emit_audio (shared_ptr<AudioBuffers> data, DCPTime time)
 {
-	/* XXX: this may not work for 3D */
-	BOOST_FOREACH (DCPTimePeriod i, subtract(period, _no_video)) {
-		for (DCPTime j = i.from; j < i.to; j += one_video_frame()) {
-			if (_playlist->video_content_at(j) && _last_video) {
-				Video (shared_ptr<PlayerVideo> (new PlayerVideo (*_last_video)), j);
-			} else {
-				Video (black_player_video_frame(), j);
-			}
-		}
-	}
+	Audio (data, time);
+	_last_audio_time = time + DCPTime::from_frames (data->frames(), _film->audio_frame_rate());
 }
 
 void
 Player::fill_audio (DCPTimePeriod period)
 {
-	BOOST_FOREACH (DCPTimePeriod i, subtract(period, _no_audio)) {
-		DCPTime t = i.from;
-		while (t < i.to) {
-			DCPTime block = min (DCPTime::from_seconds (0.5), i.to - t);
-			Frame const samples = block.frames_round(_film->audio_frame_rate());
-			if (samples) {
-				shared_ptr<AudioBuffers> silence (new AudioBuffers (_film->audio_channels(), samples));
-				silence->make_silent ();
-				Audio (silence, t);
-			}
-			t += block;
+	if (period.from == period.to) {
+		return;
+	}
+
+	DCPOMATIC_ASSERT (period.from < period.to);
+
+	DCPTime t = period.from;
+	while (t < period.to) {
+		DCPTime block = min (DCPTime::from_seconds (0.5), period.to - t);
+		Frame const samples = block.frames_round(_film->audio_frame_rate());
+		if (samples) {
+			shared_ptr<AudioBuffers> silence (new AudioBuffers (_film->audio_channels(), samples));
+			silence->make_silent ();
+			emit_audio (silence, t);
 		}
+		t += block;
 	}
 }
 
@@ -944,3 +943,29 @@ Player::one_video_frame () const
 {
 	return DCPTime::from_frames (1, _film->video_frame_rate ());
 }
+
+pair<shared_ptr<AudioBuffers>, DCPTime>
+Player::discard_audio (shared_ptr<const AudioBuffers> audio, DCPTime time, DCPTime discard_to) const
+{
+	DCPTime const discard_time = discard_to - time;
+	Frame const discard_frames = discard_time.frames_round(_film->audio_frame_rate());
+	Frame remaining_frames = audio->frames() - discard_frames;
+	if (remaining_frames <= 0) {
+		return make_pair(shared_ptr<AudioBuffers>(), DCPTime());
+	}
+	shared_ptr<AudioBuffers> cut (new AudioBuffers (audio->channels(), remaining_frames));
+	cut->copy_from (audio.get(), remaining_frames, discard_frames, 0);
+	return make_pair(cut, time + discard_time);
+}
+
+void
+Player::set_dcp_decode_reduction (optional<int> reduction)
+{
+	if (reduction == _dcp_decode_reduction) {
+		return;
+	}
+
+	_dcp_decode_reduction = reduction;
+	_have_valid_pieces = false;
+	Changed (false);
+}