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

[Paul B Mahol]

On 8/4/17, Paul B Mahol <onemda at gmail.com> wrote:
> 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.
>

Also what about to porting one of simple filter(just filter_frame
call) to activate API,
i can not do it myself unless someone show me how.