tree 2aed7225d841d40b80dcdbbae830105bad9704a4
parent 1eec6166d881981ec119b607bb4fa5c439ca9419
author Urvang Joshi <urvang@google.com> 1714070125 +0000
committer Krishna Rapaka <krapaka@apple.com> 1714070125 +0000

Use per-channel array for blk_skip to avoid uninitialized values

The old `blk_skip` array was used as a bit-field, combining blk_skip values of 3 planes into one value. However, for sub-8x8 blocks, where a chroma block can cover multiple luma blocks, the `blk_skip` values for chroma planes were uninitialized / garbage during RDO in some cases, because saving/restoring logic was only using the luma block size.

Also, added back an assertion which checked for uninitialized values -- it was removed during adoption of SDP.

The issue was observed in lossless mode only for now (see #160 ), where all transform sizes are forced to be 4x4. 

No change in stats for lossy encoding in release build.

Fixes #160