[FFmpeg-cvslog] mov: Remove duration-of-last-frame heuristic hack

Derek Buitenhuis git at videolan.org
Wed Jan 2 17:51:45 EET 2019


ffmpeg | branch: master | Derek Buitenhuis <derek.buitenhuis at gmail.com> | Mon Dec 24 14:47:55 2018 -0500| [8f585c3c550c2dd7b2a3575f3544a3bede28eb9f] | committer: Derek Buitenhuis

mov: Remove duration-of-last-frame heuristic hack

This breaks totally valid files that get caught in its heuristic.

This, according to the commit message, is my own doing, having asked
Michael to implement this check and providing a sample that was
"wrong". I am now atoning for my sins, and removing this hack, having
seen the light (aka that this was silly to do in the first place).

Resotores correct behavior on valid files.

This reverts commit 8e5e84c2a2a21a979b48e80c5a8dd44754ab3f21.

Signed-off-by: Derek Buitenhuis <derek.buitenhuis at gmail.com>

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=8f585c3c550c2dd7b2a3575f3544a3bede28eb9f
---

 libavformat/mov.c | 6 ------
 1 file changed, 6 deletions(-)

diff --git a/libavformat/mov.c b/libavformat/mov.c
index 825738127b..970cd87f70 100644
--- a/libavformat/mov.c
+++ b/libavformat/mov.c
@@ -2923,12 +2923,6 @@ static int mov_read_stts(MOVContext *c, AVIOContext *pb, MOVAtom atom)
         av_log(c->fc, AV_LOG_TRACE, "sample_count=%d, sample_duration=%d\n",
                 sample_count, sample_duration);
 
-        if (   i+1 == entries
-            && i
-            && sample_count == 1
-            && total_sample_count > 100
-            && sample_duration/10 > duration / total_sample_count)
-            sample_duration = duration / total_sample_count;
         duration+=(int64_t)sample_duration*(uint64_t)sample_count;
         total_sample_count+=sample_count;
     }



More information about the ffmpeg-cvslog mailing list