On Fri, Jul 14, 2017 at 5:38 PM, Michael Niedermayer <mich...@niedermayer.cc > wrote:
> > 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: https://chromium-review.googlesource.com/c/572730/ https://chromium-review.googlesource.com/c/572585/ 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. - dale _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
