From 0e86a5da67451dfc6120406e5282fc786de6e248 Mon Sep 17 00:00:00 2001 From: Carl Hetherington Date: Tue, 7 Jan 2020 13:30:56 +0100 Subject: [PATCH] Extend the amount of allowed disagreement between the audio coming from the content and the calculated periods of silence. It sees that there is some scope for disagreements here when trim start/end times are rounded to frame boundaries. Fixes #1708. --- src/lib/player.cc | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/lib/player.cc b/src/lib/player.cc index baf9b2068..fe08241ab 100644 --- a/src/lib/player.cc +++ b/src/lib/player.cc @@ -639,7 +639,8 @@ Player::pass () remove a little silence at the end of some content. */ int64_t const error = labs(period.from.get() - _last_audio_time->get()); - int64_t const too_much_error = 4; + /* Let's not worry about less than a frame at 24fps */ + int64_t const too_much_error = DCPTime::from_frames(1, 24).get(); if (error >= too_much_error) { _film->log()->log(String::compose("Silence starting before or after last audio by %1", error), LogEntry::TYPE_ERROR); } -- 2.30.2