[FFmpeg-devel] [PATCH] lavf/mov: add support for sidx fragment indexes
dalecurtis at chromium.org
Sat Jul 15 04:31:25 EEST 2017
On Fri, Jul 14, 2017 at 5:38 PM, Michael Niedermayer <michael at niedermayer.cc
> I dont think CTTS is the only affected atom.
Thanks for the response. Yes, I think that's likely true. The ctts one is
just likely the most important since it controls timestamp ordering.
> what you describe sounds like an option. but i might be missing something,
> possibly even something significant. I dont feel that confident with
> this code after quickly looking at it.
Here are my experiments thus far if you have further thoughts:
Both fixes and the existing code seem to suffer from assuming that the
sample number in the AVIndex and that of the ctts_data are the same; which
seems questionable. The second experiment should be viable with the
addition of memmove() when inserting in the middle of the array. Will look
at it more next week.
More information about the ffmpeg-devel