[FFmpeg-devel] [PATCH v2] Allow 'concat' filter to support inputs with different frame rates

Michael Niedermayer michael at niedermayer.cc
Sat Mar 11 15:28:29 EET 2017


On Thu, Mar 09, 2017 at 03:27:16PM -0500, Calvin Walton wrote:
> Right now, the concat filter does not set the frame_rate value on any of
> the out links. As a result, the default ffmpeg behaviour kicks in - to
> copy the framerate from the first input to the outputs.
> 
> If a later input is higher framerate, this results in dropped frames; if
> a later input is lower framerate it might cause judder.
> 
> This patch checks if all of the video inputs have the same framerate, and
> if not it sets the out link to use '1/0' as the frame rate, the value
> meaning "unknown/vfr".
> 
> A test is added to verify the VFR behaviour. The existing test for CFR
> behaviour passes unchanged.
> ---
>  libavfilter/avf_concat.c         |  15 ++-
>  tests/fate/filter-video.mak      |   4 +-
>  tests/ref/fate/filter-concat-vfr | 224 +++++++++++++++++++++++++++++++++++++++
>  3 files changed, 241 insertions(+), 2 deletions(-)
>  create mode 100644 tests/ref/fate/filter-concat-vfr

seem to fail on x86-32

make: *** No rule to make target `tests/data/filtergraphs/concat-vfr', needed by `fate-filter-concat-vfr'.  Stop.
make: *** Waiting for unfinished jobs....


[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

The misfortune of the wise is better than the prosperity of the fool.
-- Epicurus
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20170311/e456aae7/attachment.sig>


More information about the ffmpeg-devel mailing list