[FFmpeg-devel] [RFC] Request for pixdesc API review

[Stefano Sabatini]

On date Monday 2009-11-16 00:43:02 +0100, Stefano Sabatini encoded:
> On date Sunday 2009-11-08 18:35:43 +0100, Stefano Sabatini encoded:
[...]
> * In order to prove that the API is expressive enough, I rewrote
>   ff_fill_linesize() and ff_fill_pointer() in imgconvert.c to make use
>   of pixdescs, the experiment resulted in code more generic and
>   functionally equivalent to the old one (it requires some patches I
>   posted today), sample patch attached.

Some more consideration about expressivity.

PixFmtInfo contains the color_type value which can assume these
values:
FF_COLOR_RGB
FF_COLOR_GRAY
FF_COLOR_YUV
FF_COLOR_YUV_JPEG
FF_PIXEL_PLANAR
FF_PIXEL_PACKED

we came to the conclusion that colorspace information don't belong to
pixdesc, so some of these informations cannot be put in the pixdesc,
we could still add some macros in imgconvert.c and use them and still
be able to dump PixFmtInfo, would be this acceptable?

Similarly for the PixFmtInfo.is_alpha field we could either use a
PIX_FMT_ALPHA flag (not sure this is a good idea, afterall this may be
considered as a colorspace info) or use some macro.

Comments are welcome.
-- 
FFmpeg = Fast Fundamental Murdering Problematic Earthshaking God