[FFmpeg-devel] [PATCH v3 1/2] checkasm: add test for fdct
James Almer
jamrial at gmail.com
Mon May 13 19:49:02 EEST 2024
On 5/6/2024 2:49 PM, Rémi Denis-Courmont wrote:
> Le maanantaina 6. toukokuuta 2024, 20.18.11 EEST Ramiro Polla a écrit :
>> I'll send a similar patch to fix checkasm/idctdsp after this is merged.
>
> The idctdsp test does not actually test the iDCT, but only the trivial-ish
> add/put helpers, so it does not care about the context. You're welcome to fix
> it anyway of course.
I personally find it ugly how we're storing a whole AVCodecContext on
stack in these tests just to pass two ints to an init function.
Maybe we can make said values be input parameters for these instead of a
pointer to avctx.
More information about the ffmpeg-devel
mailing list