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

[Paul B Mahol]

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.