From 452a21994f6268faa159e53ead722a5bd5125b29 Mon Sep 17 00:00:00 2001 From: Carl Hetherington Date: Tue, 30 May 2017 10:11:01 +0100 Subject: [PATCH] Remove call to fill_audio which goes wrong with multi-audio-stream content. This call just doesn't take multi-stream into account. Removing it does not trip any tests so I think it may be superfluous... --- src/lib/player.cc | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/src/lib/player.cc b/src/lib/player.cc index 95ff55bc5..de221fef3 100644 --- a/src/lib/player.cc +++ b/src/lib/player.cc @@ -743,12 +743,6 @@ Player::audio_transform (shared_ptr content, AudioStreamPtr stream content_audio.audio = _audio_processor->run (content_audio.audio, _film->audio_channels ()); } - /* Pad any gap which may be caused by audio delay */ - - if (_last_audio_time) { - fill_audio (DCPTimePeriod (*_last_audio_time, time)); - } - /* Push */ _audio_merger.push (content_audio.audio, time); @@ -950,6 +944,8 @@ Player::fill_audio (DCPTimePeriod period) return; } + DCPOMATIC_ASSERT (period.from < period.to); + BOOST_FOREACH (DCPTimePeriod i, subtract(period, _no_audio)) { DCPTime t = i.from; while (t < i.to) { -- 2.30.2