Paul B Mahol (12021-07-25):
> Signed-off-by: Paul B Mahol <one...@gmail.com>
> ---
>  libavfilter/avf_concat.c | 5 ++++-
>  1 file changed, 4 insertions(+), 1 deletion(-)
> 
> diff --git a/libavfilter/avf_concat.c b/libavfilter/avf_concat.c
> index cb46f52a04..8bb5a5d618 100644
> --- a/libavfilter/avf_concat.c
> +++ b/libavfilter/avf_concat.c
> @@ -402,7 +402,10 @@ static int activate(AVFilterContext *ctx)
>              if (ret > 0) {
>                  close_input(ctx, cat->cur_idx + i);
>                  if (cat->cur_idx + ctx->nb_outputs >= ctx->nb_inputs) {

> -                    ff_outlink_set_status(ctx->outputs[i], status, pts);
> +                    ff_outlink_set_status(ctx->outputs[i], status,
> +                                          cat->delta_ts + av_rescale_q(pts,
> +                                                                       
> ctx->inputs[cat->cur_idx + i]->time_base,
> +                                                                       
> ctx->outputs[i]->time_base));

It looks valid, good catch, but ugly. Please add a local variable for
ctx->inputs[cat->cur_idx + i], which is now used twice, and re-affect pts
in the line before rather than inlining the computation.

>                  }
>                  if (!cat->nb_in_active) {
>                      ret = flush_segment(ctx);

-- 
  Nicolas George

Attachment: signature.asc
Description: PGP signature

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Reply via email to