diff options
author | Stefano Sabatini <stefasab@gmail.com> | 2013-04-15 20:28:57 +0400 |
---|---|---|
committer | Stefano Sabatini <stefasab@gmail.com> | 2013-04-18 00:28:56 +0400 |
commit | 565c50ac7b51112032cd48e81af4560cda72b5b0 (patch) | |
tree | 4b74cf53370225b90ee3c6ec5fc52e0e70e8fdfd /libavfilter/f_select.c | |
parent | 0b3bd78714ab7ae33700caa487533d9ad2f75d13 (diff) |
lavfi/select: make select_frame() return void
Set select->select internally to the function, avoid lossy double->int
conversion causing spurious frame drop (e.g. in case the evaluation
result is positive and less than 1).
Diffstat (limited to 'libavfilter/f_select.c')
-rw-r--r-- | libavfilter/f_select.c | 8 |
1 files changed, 3 insertions, 5 deletions
diff --git a/libavfilter/f_select.c b/libavfilter/f_select.c index 9a5666f809..ca9bdc9533 100644 --- a/libavfilter/f_select.c +++ b/libavfilter/f_select.c @@ -247,7 +247,7 @@ static double get_scene_score(AVFilterContext *ctx, AVFrame *frame) #define D2TS(d) (isnan(d) ? AV_NOPTS_VALUE : (int64_t)(d)) #define TS2D(ts) ((ts) == AV_NOPTS_VALUE ? NAN : (double)(ts)) -static int select_frame(AVFilterContext *ctx, AVFrame *frame) +static void select_frame(AVFilterContext *ctx, AVFrame *frame) { SelectContext *select = ctx->priv; AVFilterLink *inlink = ctx->inputs[0]; @@ -284,7 +284,7 @@ static int select_frame(AVFilterContext *ctx, AVFrame *frame) break; } - res = av_expr_eval(select->expr, select->var_values, NULL); + select->select = res = av_expr_eval(select->expr, select->var_values, NULL); av_log(inlink->dst, AV_LOG_DEBUG, "n:%f pts:%f t:%f key:%d", select->var_values[VAR_N], @@ -322,15 +322,13 @@ static int select_frame(AVFilterContext *ctx, AVFrame *frame) select->var_values[VAR_N] += 1.0; select->var_values[VAR_PREV_PTS] = select->var_values[VAR_PTS]; select->var_values[VAR_PREV_T] = select->var_values[VAR_T]; - - return res; } static int filter_frame(AVFilterLink *inlink, AVFrame *frame) { SelectContext *select = inlink->dst->priv; - select->select = select_frame(inlink->dst, frame); + select_frame(inlink->dst, frame); if (select->select) return ff_filter_frame(inlink->dst->outputs[0], frame); |