[FFmpeg-devel] [PATCH 07/14] lavfi/vf_overlay: use framesync2 options.

Paul B Mahol onemda at gmail.com
Fri Aug 4 21:06:23 EEST 2017

On 8/2/17, Nicolas George <george at nsup.org> wrote:
> Le quintidi 15 thermidor, an CCXXV, Michael Niedermayer a écrit :
>> removing these elements causes command lines using them to fail
>> for example:
>> ./ffmpeg -i matrixbench_mpeg2.mpg -i lena.pnm -filter_complex
>> 'overlay=10:main_h-overlay_h-10:pass' -qscale 2 -t 1 file.avi
> Oh, the shorthand notation. The order of the non-essential options has
> changed in the past, including recently, and does not match the
> documentation, I have always considered it undocumented and as such
> unspecified.
> Moreover, with a separate object like that, keeping the order will be
> impossible: options added to the filter in the future will always come
> before the options in the framesync object.
> Therefore, I consider this minor break of compatibility acceptable. I
> should have mentioned it, but I forgot, sorry.
> What do other think?

We should move forward, no need to keep shorthand compatibility for
those options.

More information about the ffmpeg-devel mailing list