[FFmpeg-devel] [PATCH] lavf/mov: add support for sidx fragment indexes

Rodger Combs rodger.combs at gmail.com
Fri Feb 12 11:21:44 CET 2016


This issue is fixed by this patch, but I'm unsure of possible implications on other files. It passes FATE, at least.

diff --git a/libavformat/mov.c b/libavformat/mov.c
index 149e3b4..c5e0a1e 100644
--- a/libavformat/mov.c
+++ b/libavformat/mov.c
@@ -3609,7 +3609,7 @@ static int mov_read_trun(MOVContext *c, AVIOContext *pb, MOVAtom atom)
                 }
                 av_log(c->fc, AV_LOG_DEBUG, "calculated into dts %"PRId64"\n", dts);
             } else {
-                dts = frag->time;
+                dts = frag->time - sc->time_offset;
                 av_log(c->fc, AV_LOG_DEBUG, "found frag time %"PRId64
                         ", using it for dts\n", dts);
             }


> On Jan 15, 2016, at 16:57, Michael Niedermayer <michael at niedermayer.cc> wrote:
> 
> On Fri, Jan 15, 2016 at 10:24:43PM +0000, Dan Sanders wrote:
>> Michael, I wanted to check if you have you looked into this playback issue,
>> or were planning to?
> 
> i didnt look into it, i had thought rodger would look into it as it
> was his patch ...
> 
> rodger, did you look into this ?
> 
> [...]
> -- 
> Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
> 
> Rewriting code that is poorly written but fully understood is good.
> Rewriting code that one doesnt understand is a sign that one is less smart
> then the original author, trying to rewrite it will not make it better.



More information about the ffmpeg-devel mailing list