[FFmpeg-devel] [PATCH 2/2] avfilter/f_loop: warn about unset loop sizes

Paul B Mahol onemda at gmail.com
Fri May 24 11:01:14 EEST 2019


On 5/23/19, Marton Balint <cus at passwd.hu> wrote:
> Signed-off-by: Marton Balint <cus at passwd.hu>
> ---
>  libavfilter/f_loop.c | 13 +++++++++++++
>  1 file changed, 13 insertions(+)
>
> diff --git a/libavfilter/f_loop.c b/libavfilter/f_loop.c
> index fcbd742eb4..5ec44d9da2 100644
> --- a/libavfilter/f_loop.c
> +++ b/libavfilter/f_loop.c
> @@ -55,6 +55,15 @@ typedef struct LoopContext {
>  #define VFLAGS AV_OPT_FLAG_VIDEO_PARAM|AV_OPT_FLAG_FILTERING_PARAM
>  #define OFFSET(x) offsetof(LoopContext, x)
>
> +static void check_size(AVFilterContext *ctx)
> +{
> +    LoopContext *s = ctx->priv;
> +
> +    if (!s->size)
> +        av_log(ctx, AV_LOG_WARNING, "Number of %s to loop is not set!\n",
> +               ctx->input_pads[0].type == AVMEDIA_TYPE_VIDEO ? "frames" :
> "samples");
> +}
> +
>  #if CONFIG_ALOOP_FILTER
>
>  static int aconfig_input(AVFilterLink *inlink)
> @@ -67,6 +76,8 @@ static int aconfig_input(AVFilterLink *inlink)
>      if (!s->fifo || !s->left)
>          return AVERROR(ENOMEM);
>
> +    check_size(ctx);
> +
>      return 0;
>  }
>
> @@ -250,6 +261,8 @@ static av_cold int init(AVFilterContext *ctx)
>      if (!s->frames)
>          return AVERROR(ENOMEM);
>
> +    check_size(ctx);
> +
>      return 0;
>  }
>

lgtm

> --
> 2.16.4
>
> _______________________________________________
> 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