On Wed, 12 Mar 2014 07:48:51 +0100
Vittorio Giovara <[email protected]> wrote:
> ---
> As noted by wm4 the full scale formats have been deprecated for ages but
> the entries of the enum have not (which this patch takes care of).
>
> Also I'm a bit puzzled that pixel formats are in lavu while color space and
> color range are in lavc. Would it be better to move them all to lavu?
Yes.
> Another point, (again AFAIU) pixel format/range/space seem to be more a frame
> property, instead of a context one (kinda like interlacing). Would it make
> sense
> to move everything there? The API I would envision would be an entry in
> AVFrame
> and an AVFrameSideData for the color range and spaces.
They could be direct fields in AVFrame. I think they're important
enough - they're crucial for correct processing or display. And there
are much less important fields in AVFrame. I would suggest adding the
following fields:
enum AVColorSpace colorspace;
enum AVColorRange color_range;
ffmpeg already has these. And I don't think there's a need for
gratuitous incompatibility without reasons.
> Opinions on this?
> Vittorio
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel
