There is no need to duplicate the input reference, since a filter should not pass along a reference which is later modified. If this happens the filter passing the reference should be fixed. Also improve consistency with start_frame() of settb, allowing a pending factorization.
Unified · Split
| @@ -116,22 +116,20 @@ static int start_frame(AVFilterLink *inlink, AVFilterBufferRef *picref) · 115 unmodified lines | ||
| 116 | 116 | return ff_start_frame(outlink, picref); |
| 117 | 117 | } |
| 118 | 118 | |
| 119 | −static int filter_samples(AVFilterLink *inlink, AVFilterBufferRef *insamples) | |
| 119 | +static int filter_samples(AVFilterLink *inlink, AVFilterBufferRef *samplesref) | |
| 120 | 120 | { |
| 121 | 121 | AVFilterContext *ctx = inlink->dst; |
| 122 | 122 | AVFilterLink *outlink = ctx->outputs[0]; |
| 123 | − AVFilterBufferRef *outsamples = insamples; | |
| 124 | 123 | |
| 125 | 124 | if (av_cmp_q(inlink->time_base, outlink->time_base)) { |
| 126 | − outsamples = avfilter_ref_buffer(insamples, ~0); | |
| 127 | − outsamples->pts = av_rescale_q(insamples->pts, inlink->time_base, outlink->time_base); | |
| 125 | + int64_t orig_pts = samplesref->pts; | |
| 126 | + samplesref->pts = av_rescale_q(samplesref->pts, inlink->time_base, outlink->time_base); | |
| 128 | 127 | av_log(ctx, AV_LOG_DEBUG, "tb:%d/%d pts:%"PRId64" -> tb:%d/%d pts:%"PRId64"\n", |
| 129 | − inlink ->time_base.num, inlink ->time_base.den, insamples ->pts, | |
| 130 | − outlink->time_base.num, outlink->time_base.den, outsamples->pts); | |
| 131 | − avfilter_unref_buffer(insamples); | |
| 128 | + inlink ->time_base.num, inlink ->time_base.den, orig_pts, | |
| 129 | + outlink->time_base.num, outlink->time_base.den, samplesref->pts); | |
| 132 | 130 | } |
| 133 | 131 | |
| 134 | − return ff_filter_samples(outlink, outsamples); | |
| 132 | + return ff_filter_samples(outlink, samplesref); | |
| 135 | 133 | } |
| 136 | 134 | |
| 137 | 135 | #if CONFIG_SETTB_FILTER |