[FFmpeg-devel] [PATCH 10/13] Add rawvideo pixel formats to codec tags mapping for the newly added nut rawvideo pixel formats codec tags.

[Stefano Sabatini]

On date Sunday 2010-05-09 23:18:47 +0200, Michael Niedermayer encoded:
> On Sun, May 09, 2010 at 03:33:27PM +0200, Stefano Sabatini wrote:
[...]
> >  nut4cc.txt |   10 ++++++++++
> >  1 file changed, 10 insertions(+)
> > 4ad3cc6b63e9431047b26f2965899ae6c9b8cefe  nut4cc-additions.patch
> > Index: docs/nut4cc.txt
> > ===================================================================
> > --- docs/nut4cc.txt	(revision 666)
> > +++ docs/nut4cc.txt	(working copy)
> > @@ -91,6 +91,16 @@
> >  ARGB    Packed RGBA 8:8:8:8, 32bpp, ARGB [NOT in AVI]
> >  RGB[24] Packed RGB 8:8:8, 24bpp RGB [NOT in AVI]
> >  BGR[24] Packed RGB 8:8:8, 24bpp BGR [NOT in AVI]
> > +MOWH    1bpp, 0 is white, 1 is black [NOT in AVI]
> > +MOBL    1bpp, 0 is black, 1 is white [NOT in AVI]
> 
> these appear fliped from what i would have expected

This is misleading but correct.
 
> 
> > +BGR[8]  Packed RGB 3:3:2,  8bpp, (msb)2B 3G 3R(lsb) [NOT in AVI]
> > +RGB[8]  Packed RGB 3:3:2,  8bpp, (msb)2B 3G 3R(lsb) [NOT in AVI]
>                                          ^^^^^^^^
> copy and paste error

Fixed.
 
> > +BGR[4]  Packed RGB 1:2:1,  4bpp, (msb)1B 2G 1R(lsb) [NOT in AVI]
> > +RGB[4]  Packed RGB 1:2:1,  4bpp, (msb)1B 2G 1R(lsb) [NOT in AVI]
> 
> ambigous, (the placeent of pixels in a byte)

This is a bitstream format, clarified.
 
> 
> > +NV12    Planar YUV 4:2:0, 12bpp, 1 plane for Y and 1 for UV [NOT in AVI]
> > +NV21    as above, but U and V bytes are swapped [NOT in AVI]
> 
> 1 for UV is not clear, what is that interleaved bits, rows ?

Clarified.

> > +RGB[48] Packed RGB 16:16:16, 48bpp, 16R, 16G, 16B, little-endian [NOT in AVI]
> > +[48]BGR Packed RGB 16:16:16, 48bpp, 16R, 16G, 16B, big-endian    [NOT in AVI]
> 
> this is not well specified
> also iam not sure if not [48]RGB would be more correct name here but
> this is debatebale, iam unsure

Yes I prefer [48]RGB as well.

Regards.
-- 
FFmpeg = Furious Frightening Maxi Prodigious Emblematic God