On 27/01/14 16:19, Vittorio Giovara wrote:
> Hi all,
> this set of patches tries to improve interlaced frames handling in libav.
> 
> First of all it updates AVFrame to use one AVFieldState variable instead of
> two different integers. Then it introduces two new "states", that is when
> the interlacing mode is unknown and when you know that the frame interlaced
> but not if it's top field first or bottom field first. This also addresses
> peculiar decoders, like png and utvideo, whose concept of interlacing is
> slightly different than the other.
> 
> Finally it sliglty modifies the context information so that when
> encoding/decoding, users can immediately see if a video contains interlaced
> frames or not, and (optionally) override interlacing coding from the command
> line.
> 
> This is my first large deprecation patchset, so please be gentle ^^.
> Vittorio


To make the transition painless you should probably have some internal
getter/setter

ff_avframe_interlace(avframe, FIELD);
ff_avframe_get_interlace(avframe, &field);

That automagically makes the flags consistent.

Then once we can move away from them we can get back setting just the
field directly.

lu

_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to