On Thu, 12 Aug 2021, Hu Weiwen wrote:

"frag_start" is redundant, and every occurance can be replaced with cluster[0].dts - start_dts

I think I can agree about this, so I think the patch should be fine, thanks for taking the time to study it in detail!

My own mental model of the code, having written it this way, is based around having this variable, but I guess I can adapt to this change too.

While the change is ok, can we defer pushing it a couple days, in case something comes up in my (or your) mind about something we forgot to think about?

// Martin

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Reply via email to