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

[Nicolas George]

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?

Regards,

-- 
  Nicolas George
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20170802/871f5985/attachment.sig>