From 520c0736fd2079f04a2a0ba09562bae49f62f472 Mon Sep 17 00:00:00 2001 From: Paul B Mahol Date: Fri, 6 Jan 2017 12:27:46 +0100 Subject: [PATCH] avfilter/vf_shuffleframes: allow also dropping frames Signed-off-by: Paul B Mahol --- doc/filters.texi | 3 ++- libavfilter/vf_shuffleframes.c | 14 ++++++++------ 2 files changed, 10 insertions(+), 7 deletions(-) diff --git a/doc/filters.texi b/doc/filters.texi index 42cdd2ea71..94548b1a1a 100644 --- a/doc/filters.texi +++ b/doc/filters.texi @@ -12285,7 +12285,7 @@ Set the size of the box used to represent one palette color entry. Default is @section shuffleframes -Reorder and/or duplicate video frames. +Reorder and/or duplicate and/or drop video frames. It accepts the following parameters: @@ -12294,6 +12294,7 @@ It accepts the following parameters: Set the destination indexes of input frames. This is space or '|' separated list of indexes that maps input frames to output frames. Number of indexes also sets maximal value that each index may have. +'-1' index have special meaning and that is to drop frame. @end table The first frame has the index 0. The default is to keep the input unchanged. diff --git a/libavfilter/vf_shuffleframes.c b/libavfilter/vf_shuffleframes.c index bfbf4bd268..8e595111b8 100644 --- a/libavfilter/vf_shuffleframes.c +++ b/libavfilter/vf_shuffleframes.c @@ -68,7 +68,7 @@ static av_cold int init(AVFilterContext *ctx) return AVERROR(EINVAL); } - if (s->map[n] < 0 || s->map[n] >= nb_items) { + if (s->map[n] < -1 || s->map[n] >= nb_items) { av_log(ctx, AV_LOG_ERROR, "Index out of range.\n"); av_free(mapping); return AVERROR(EINVAL); @@ -99,11 +99,13 @@ static int filter_frame(AVFilterLink *inlink, AVFrame *frame) AVFrame *out; x = s->map[n]; - out = av_frame_clone(s->frames[x]); - if (!out) - return AVERROR(ENOMEM); - out->pts = s->pts[n]; - ret = ff_filter_frame(ctx->outputs[0], out); + if (x >= 0) { + out = av_frame_clone(s->frames[x]); + if (!out) + return AVERROR(ENOMEM); + out->pts = s->pts[n]; + ret = ff_filter_frame(ctx->outputs[0], out); + } s->in_frames--; }