[FFmpeg-devel] [PATCH 2/6] avfilter/avf_showcqt: Mark arrays as static const

Paul B Mahol onemda at gmail.com
Sat Aug 8 17:45:51 EEST 2020


On 8/8/20, Andreas Rheinhardt <andreas.rheinhardt at gmail.com> wrote:
> Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt at gmail.com>
> ---
>  libavfilter/avf_showcqt.c | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/libavfilter/avf_showcqt.c b/libavfilter/avf_showcqt.c
> index cb0dca505f..990db3143b 100644
> --- a/libavfilter/avf_showcqt.c
> +++ b/libavfilter/avf_showcqt.c
> @@ -1321,12 +1321,12 @@ static int query_formats(AVFilterContext *ctx)
>      AVFilterChannelLayouts *layouts = NULL;
>      AVFilterLink *inlink = ctx->inputs[0];
>      AVFilterLink *outlink = ctx->outputs[0];
> -    enum AVSampleFormat sample_fmts[] = { AV_SAMPLE_FMT_FLT,
> AV_SAMPLE_FMT_NONE };
> -    enum AVPixelFormat pix_fmts[] = {
> +    static const enum AVSampleFormat sample_fmts[] = { AV_SAMPLE_FMT_FLT,
> AV_SAMPLE_FMT_NONE };
> +    static const enum AVPixelFormat pix_fmts[] = {
>          AV_PIX_FMT_YUV420P, AV_PIX_FMT_YUV422P,
>          AV_PIX_FMT_YUV444P, AV_PIX_FMT_RGB24, AV_PIX_FMT_NONE
>      };
> -    int64_t channel_layouts[] = { AV_CH_LAYOUT_STEREO,
> AV_CH_LAYOUT_STEREO_DOWNMIX, -1 };
> +    static const int64_t channel_layouts[] = { AV_CH_LAYOUT_STEREO,
> AV_CH_LAYOUT_STEREO_DOWNMIX, -1 };
>      int ret;
>
>      /* set input audio formats */
> --
> 2.20.1
>

LGTM

> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request at ffmpeg.org with subject "unsubscribe".


More information about the ffmpeg-devel mailing list