On 12/03/2025 04:10, Andreas Rheinhardt wrote:
Patches attached.- Andreas
First patch is wild, its surprising no one considered inverting the way decoder parses codes for an encoder yet. Rather than ORing and using put_bits63, I think it would make more sense to write out each chunk using put_bits sequentially. It might be possible to reverse the lookups such that you get the MSBs first so you wouldn't need to reverse them out of place in a small array.
But either way, LGTM. Feel free to explore this in a follow-up.Second patch seems a bit pointless. It's just one single call you're uninlining? Chasing to save a few extra bytes of binary surely don't deserve having a wrapper function for uninlining.
OpenPGP_0xA2FEA5F03F034464.asc
Description: OpenPGP public key
OpenPGP_signature.asc
Description: OpenPGP digital signature
_______________________________________________ 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".
