Devin Heitmueller <devin.heitmuel...@ltnglobal.com> writes:
> On Wed, Apr 24, 2024 at 10:10 AM J. Dekker <j...@itanimul.li> wrote:
>>
>> Signed-off-by: J. Dekker <j...@itanimul.li>
>> ---
>> tests/checkasm/h264dsp.c | 6 ++++--
>> 1 file changed, 4 insertions(+), 2 deletions(-)
>>
>> diff --git a/tests/checkasm/h264dsp.c b/tests/checkasm/h264dsp.c
>> index 0f484e3f43..5cb646ae49 100644
>> --- a/tests/checkasm/h264dsp.c
>> +++ b/tests/checkasm/h264dsp.c
>> @@ -173,6 +173,7 @@ static void dct8x8(int16_t *coef, int bit_depth)
>>
>> static void check_idct(void)
>> {
>> + static const int depths[5] = { 8, 9, 10, 12, 14 };
>> LOCAL_ALIGNED_16(uint8_t, src, [8 * 8 * 2]);
>> LOCAL_ALIGNED_16(uint8_t, dst, [8 * 8 * 2]);
>> LOCAL_ALIGNED_16(uint8_t, dst0, [8 * 8 * 2]);
>> @@ -181,10 +182,11 @@ static void check_idct(void)
>> LOCAL_ALIGNED_16(int16_t, subcoef0, [8 * 8 * 2]);
>> LOCAL_ALIGNED_16(int16_t, subcoef1, [8 * 8 * 2]);
>> H264DSPContext h;
>> - int bit_depth, sz, align, dc;
>> + int bit_depth, sz, align, dc, i;
>> declare_func_emms(AV_CPU_FLAG_MMX, void, uint8_t *dst, int16_t *block,
>> int stride);
>>
>> - for (bit_depth = 8; bit_depth <= 10; bit_depth++) {
>> + for (i = 0; i < 5; i++) {
>> + bit_depth = depths[i];
>
> Perhaps this should use FF_ARRAY_ELEMS(depths) rather than a hard-coded "5"?
Thanks for the suggestion, pushed with this change.
--
jd
_______________________________________________
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 with subject "unsubscribe".
