> On Apr 21, 2025, at 14:04, Shiqi Zhu <hiccup...@gmail.com> wrote:
> 
> Signed-off-by: Shiqi Zhu <hiccup...@gmail.com>
> ---
> libavfilter/af_volume.c | 18 ++++++------------
> 1 file changed, 6 insertions(+), 12 deletions(-)
> 
> diff --git a/libavfilter/af_volume.c b/libavfilter/af_volume.c
> index 471bffeceb..c16188524d 100644
> --- a/libavfilter/af_volume.c
> +++ b/libavfilter/af_volume.c
> @@ -178,34 +178,28 @@ static inline void scale_samples_u8_small(uint8_t *dst, 
> const uint8_t *src,
>         dst[i] = av_clip_uint8((((src[i] - 128) * volume + 128) >> 8) + 128);
> }
> 
> -static inline void scale_samples_s16(uint8_t *dst, const uint8_t *src,
> +static inline void scale_samples_s16(int16_t *dst, const int16_t *src,
>                                      int nb_samples, int volume)
> {
>     int i;
> -    int16_t *smp_dst       = (int16_t *)dst;
> -    const int16_t *smp_src = (const int16_t *)src;
>     for (i = 0; i < nb_samples; i++)
> -        smp_dst[i] = av_clip_int16(((int64_t)smp_src[i] * volume + 128) >> 
> 8);
> +        dst[i] = av_clip_int16(((int64_t)src[i] * volume + 128) >> 8);
> }
> 
> -static inline void scale_samples_s16_small(uint8_t *dst, const uint8_t *src,
> +static inline void scale_samples_s16_small(int16_t *dst, const int16_t *src,
>                                            int nb_samples, int volume)
> {
>     int i;
> -    int16_t *smp_dst       = (int16_t *)dst;
> -    const int16_t *smp_src = (const int16_t *)src;
>     for (i = 0; i < nb_samples; i++)
> -        smp_dst[i] = av_clip_int16((smp_src[i] * volume + 128) >> 8);
> +        dst[i] = av_clip_int16((src[i] * volume + 128) >> 8);
> }
> 
> -static inline void scale_samples_s32(uint8_t *dst, const uint8_t *src,
> +static inline void scale_samples_s32(int32_t *dst, const int32_t *src,
>                                      int nb_samples, int volume)
> {
>     int i;
> -    int32_t *smp_dst       = (int32_t *)dst;
> -    const int32_t *smp_src = (const int32_t *)src;
>     for (i = 0; i < nb_samples; i++)
> -        smp_dst[i] = av_clipl_int32((((int64_t)smp_src[i] * volume + 128) >> 
> 8));
> +        dst[i] = av_clipl_int32((((int64_t)src[i] * volume + 128) >> 8));
> }
> 
> static av_cold void volume_init(VolumeContext *vol)

NAK. They are incompatible function pointer types.

> -- 
> 2.34.1
> 
> _______________________________________________
> 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".

Reply via email to