On Fri, 21 Mar 2014 12:56:26 +0100 Vittorio Giovara <[email protected]> wrote:
> On Friday, March 21, 2014, Kieran Kunhya <[email protected]> wrote: > > > > +2014-xx-xx - xxxxxxx - lavu 53.07.0 - pixmodel.h > > > + Move all color-related enums (AVColorPrimaries, AVColorSpace, > > AVColorRange, > > > + and AVChromaLocation) inside lavu, and introduce a new type > > containing them. > > > + > > > > Except colour information is an inherent part of the decoded video and > > is not "side-data" by any definition. > > > My understanding of side data is that anything that is not essential to the > decoding phase is a side data. Pixel_format is essential because it tells > how bits are packed, thus it's an AVFrame field; however range, primaries > etc are often used after decoding, for rendering or scaling the picture, > and the decoded picture is usable even if that information is missing. I'd prefer another definition: everything that is commonly needed to interpret the data should be directly in the AVFrame. Your change would just lead to every single video frame having this color side data, which is not really the purpose of side data. Note that current AVFrame has a lot of information that is not strictly needed for "decoding", like sample rate or pts. _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
