On Mon, Sep 9, 2024 at 8:04 PM Zhao Zhili <quinkbl...@foxmail.com> wrote:
>
>
> > On Sep 9, 2024, at 18:35, Anton Khirnov <an...@khirnov.net> wrote:
> >
> > Quoting Zhao Zhili (2024-09-07 19:13:40)
> >> From: Zhao Zhili <zhiliz...@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.
>
> They are accessed as const int8_t * in h256_inter_template.c. For example
>
> static void
> put_hevc_pel_pixels_8(int16_t *dst, const uint8_t *src, ptrdiff_t
> srcstride,
> int height, intptr_t mx, intptr_t my, int width) {
> const uint8_t *hf = ff_hevc_qpel_filters[mx];
> const uint8_t *vf = ff_hevc_qpel_filters[my];
> put_pixels_8(dst, src, srcstride, height, hf, vf, width);
> }
>
> I experienced a lot of headaches with these macro.
>
Yeah.
Difficult to find, difficult to read, and difficult to understand.
>
> >
> > --
> > Anton Khirnov
> > _______________________________________________
> > 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".
>
> _______________________________________________
> 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".
>
_______________________________________________
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".
