[FFmpeg-devel] [PATCH] lavfi/overlay: add format option
stefasab at gmail.com
Wed Feb 20 00:23:20 CET 2013
On date Tuesday 2013-02-19 23:12:56 +0100, Nicolas George encoded:
> Le primidi 1er ventôse, an CCXXI, Stefano Sabatini a écrit :
> > In particular, fix misbehavior in case main and overlay input formats
> > mismatch (e.g. YUV420 and YUV444).
> It looks like of lot of code for something that can be done just adding the
> format filter after the overlay filter.
This can't work, since I need to know the output when doing
query_formats(), and set main and overlay input accordingly.
> Can you elaborate on why you chose to implement that?
This is done by extending the logic already there (rgb option) and
fixing the YUV444/YUV420 mismatch. I suppose the next step would be to
add dedicated per-format FATE tests.
Note that a possibly better fix would be to rescale the main and
overlay input to the output format, forced with another mechanism, but
this would add a dependency on libswscale and would possibly
complicate the logic, thus the current hack is good enough for me.
Feel free to propose a better fix.
FFmpeg = Free & Fantastic Maxi Political Encoding/decoding Genius
More information about the ffmpeg-devel