[FFmpeg-devel] [PATCH 3/3] lavfi/overlay: support timeline through the new system.

Stefano Sabatini stefasab at gmail.com
Sat Apr 20 11:49:24 CEST 2013


On date Friday 2013-04-19 00:28:54 +0200, Clément Bœsch encoded:
> ---
>  doc/filters.texi         | 15 +++---------
>  libavfilter/vf_overlay.c | 64 +++++++++++++++++++++++++-----------------------
>  2 files changed, 37 insertions(+), 42 deletions(-)
[...]

I see two issues with this patch:
- some variables are not supported in the generic enable expression
- enable command is removed

The First problem should be mostly addressed by adding pos to the list
of supported variables. You may ask, why is pos relevant? Well if you
know the total size of a file pos can provide some hints about the
position of the frame in the file (so you can roughly say: enable this
if it is between 50% and 60% of the file).

The second problem is more serious, indeed the enable command was the
main reason for which I added that option.
-- 
FFmpeg = Fundamental and Fanciful Moronic Political Eager Game


More information about the ffmpeg-devel mailing list