On 01/10/2019 18:25, James Almer wrote: > The value in the unused field will be 0xFFFFFFFF after this change > instead of 0, since you're writing 32 bits as duration instead of 64 > where the high 32 bits (corresponding to the unused field) are zeroed. > That means the ivf demuxer prior to this patch will read bogus duration > values from ivf files created after this patch. > > Just leave the muxer as is.
Why not just write zero? It's, to me, worse to leave a bogus 64-bit write to appease bugs in our own demuxer. It's confusing and misleading for any readers of the code. - Derek _______________________________________________ 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".
