The module suffers a lot from bad code. It's mildly complex but due to missing abstractions/separations the complexity is spread throughout code of completely different concerns. This makes it hard to change/review so people tend to stuff new code into the existing mess.
3 concrete points:
Reduce code duplication for different branches that simply handle different array lengths.
Turn the implicit smallpad, medpad, largepad layouts into structs and use those (combined with casts) to avoid all the void* arithmetic.
Turn the block info cache into a struct.
Comment #1 by robert.schadek — 2024-12-07T13:34:37Z