On Sat, 13 Aug 2022, Swinney, Jonathan wrote:
We don't generally use stdbool in ffmpeg, even if it's C99 - just use a
plain int and 0/1.
Updated this.
Other than that, the checkasm changes look fine (I coauthored part of
them - and your cleanup of my WIP patch looks good!).
Yes, thank you for the help on that!
Hmm, can you elaborate on this bit? With only the first patch applied, the
checkasm test still succeeds.
That was leftover from when my test was broken. Is that that with the fixed
test, it works fine.
Could we split this improvement for the existing codepath into a separate
preceding patch, to keep things a bit clearer?
I split it out. Let me know if I didn't split like you intended.
Thanks, this looked good to me, so I pushed them!
// 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".
