[FFmpeg-devel] [PATCH 6/6] avcodec/hevc: ff_hevc_(qpel/epel)_filters are signed type

Anton Khirnov anton at khirnov.net
Mon Sep 9 13:35:00 EEST 2024


Quoting Zhao Zhili (2024-09-07 19:13:40)
> From: Zhao Zhili <zhilizhao at tencent.com>
> 
> ---
>  libavcodec/hevc/dsp_template.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/libavcodec/hevc/dsp_template.c b/libavcodec/hevc/dsp_template.c
> index aebccd1a0c..a0f79c2673 100644
> --- a/libavcodec/hevc/dsp_template.c
> +++ b/libavcodec/hevc/dsp_template.c
> @@ -302,8 +302,8 @@ IDCT_DC(32)
>  ////////////////////////////////////////////////////////////////////////////////
>  #define ff_hevc_pel_filters ff_hevc_qpel_filters
>  #define DECL_HV_FILTER(f)                              \
> -    const uint8_t *hf = ff_hevc_ ## f ## _filters[mx]; \
> -    const uint8_t *vf = ff_hevc_ ## f ## _filters[my];
> +    const int8_t *hf = ff_hevc_ ## f ## _filters[mx]; \
> +    const int8_t *vf = ff_hevc_ ## f ## _filters[my];

Looks ok, though I wonder why are these then used as intptr_t.

-- 
Anton Khirnov


More information about the ffmpeg-devel mailing list