[FFmpeg-devel] [PATCH] swresample/swresample: Remove array size hint from swr_convert()

Andreas Rheinhardt andreas.rheinhardt at outlook.com
Thu Dec 2 08:40:15 EET 2021


Andreas Rheinhardt:
> SWR_CH_MAX is internal only and the arrays are therefore not required
> to have that many elements (and they typically don't do it). So remove
> this potentially confusing hint.
> 
> (Newer versions of GCC emit -Warray-parameter= warnings for this,
> because the definition with explicit size differs from the declaration
> (which leaves the size unspecified); this is IMO a false-positive,
> because definition and declaration didn't conflict, but anyway it is
> fixed by this commit.)
> 
> Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt at outlook.com>
> ---
>  libswresample/swresample.c | 6 ++++--
>  1 file changed, 4 insertions(+), 2 deletions(-)
> 
> diff --git a/libswresample/swresample.c b/libswresample/swresample.c
> index 22c2c33673..c03fe5528f 100644
> --- a/libswresample/swresample.c
> +++ b/libswresample/swresample.c
> @@ -711,8 +711,10 @@ int swr_is_initialized(struct SwrContext *s) {
>      return !!s->in_buffer.ch_count;
>  }
>  
> -int attribute_align_arg swr_convert(struct SwrContext *s, uint8_t *out_arg[SWR_CH_MAX], int out_count,
> -                                                    const uint8_t *in_arg [SWR_CH_MAX], int  in_count){
> +int attribute_align_arg swr_convert(struct SwrContext *s,
> +                                          uint8_t **out_arg, int out_count,
> +                                    const uint8_t **in_arg,  int in_count)
> +{
>      AudioData * in= &s->in;
>      AudioData *out= &s->out;
>      int av_unused max_output;
> 

Will apply tonight unless there are objections.

- Andreas


More information about the ffmpeg-devel mailing list