[FFmpeg-devel] [FFmpeg-cvslog] avfilter/vf_xfade: check that fps between inputs are valid

Nicolas George george at nsup.org
Tue Aug 11 12:11:09 EEST 2020


Paul B Mahol (12020-08-11):
> ffmpeg | branch: master | Paul B Mahol <onemda at gmail.com> | Tue Aug 11 10:55:16 2020 +0200| [4dbb75c437117285cfa195c437a8a1ee898e9293] | committer: Paul B Mahol
> 
> avfilter/vf_xfade: check that fps between inputs are valid
> 
> > http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=4dbb75c437117285cfa195c437a8a1ee898e9293
> ---
> 
>  libavfilter/vf_xfade.c | 16 ++++++++++++++++
>  1 file changed, 16 insertions(+)

This should not have been pushed. If it had been posted for review, I
would have had a chance to explain to you that filters are not supposed
to use information from the frame rate, as it is not negotiated and
supposed to be only indicative.

Filters that need matching timestamps between frames on separate streams
should use framesync or an equivalent logic.

-- 
  Nicolas George
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20200811/51b8f346/attachment.sig>


More information about the ffmpeg-devel mailing list