[FFmpeg-devel] [PATCH v3 1/2] checkasm: add test for fdct

Ramiro Polla ramiro.polla at gmail.com
Mon May 6 20:18:11 EEST 2024


On Thu, May 2, 2024 at 8:05 PM Rémi Denis-Courmont <remi at remlab.net> wrote:
> Le keskiviikkona 17. huhtikuuta 2024, 21.01.37 EEST Ramiro Polla a écrit :
[...]
> > +static void check_fdct(void)
> > +{
> > +    LOCAL_ALIGNED_16(int16_t, block0, [64]);
> > +    LOCAL_ALIGNED_16(int16_t, block1, [64]);
> > +
> > +    AVCodecContext avctx = { 0 };
>
> AFAICT, that is not a legal context for ff_fdctdst_init(), which expect
> bits_per_raw_sample to be one of 8, 9 or 10. It would also be good manners to
> initialise dct_algo.

Thanks for spotting it. New patch coming up in a while.

I'll send a similar patch to fix checkasm/idctdsp after this is merged.


More information about the ffmpeg-devel mailing list