Re: [FFmpeg-devel] [PATCH 5/5] avcodec/cfhddata: Reduce stack usage

2022-09-05 Thread Paul B Mahol
On Sat, Sep 3, 2022 at 11:56 PM Andreas Rheinhardt < andreas.rheinha...@outlook.com> wrote: > Paul B Mahol: > > The FATE does not cover 9 (old) codebook, so make sure it is still > working. > > > > It's CRC checksum didn't change in patches 2-5. > Set LGTM. > > - Andreas > _

Re: [FFmpeg-devel] [PATCH 5/5] avcodec/cfhddata: Reduce stack usage

2022-09-03 Thread Andreas Rheinhardt
Paul B Mahol: > The FATE does not cover 9 (old) codebook, so make sure it is still working. > It's CRC checksum didn't change in patches 2-5. - Andreas ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-dev

Re: [FFmpeg-devel] [PATCH 5/5] avcodec/cfhddata: Reduce stack usage

2022-09-03 Thread Paul B Mahol
The FATE does not cover 9 (old) codebook, so make sure it is still working. ___ 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

[FFmpeg-devel] [PATCH 5/5] avcodec/cfhddata: Reduce stack usage

2022-09-03 Thread Andreas Rheinhardt
Creating CFHD RL VLC tables works by first extending the codes by the sign, followed by creating a VLC, followed by deriving the RL VLC from this VLC (which is then discarded). Extending the codes uses stack arrays. The tables used to initialize the VLC are already sorted from left-to-right in the