[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 Wednesday 2010-05-12 02:26:14 +0200, Michael Niedermayer encoded:
> On Wed, May 12, 2010 at 01:31:26AM +0200, Stefano Sabatini wrote:
> > 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.
> 
> in ffmpeg yes, in nut its undefined and we can define it as we prefer
> and i dont like knowingly adding something we consider misleading

OK have a look at my idea, I don't want to use anymore the terms
monowhite/monoblack with a different meaning from that of pixfmt.h, to
avoid confusion to the FFmpeg user.
 
> > > > +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.
> 
> bitstream doesnt clarify it
> the msb or the lsb can be first in a byte in a bitstream

No I cannot get this, please elaborate.

Update nut4cc.txt patch attached.

Regards.
-- 
FFmpeg = Fabulous & Fiendish Maxi Pitiless Exxagerate Goblin
-------------- next part --------------
A non-text attachment was scrubbed...
Name: extend-nut-fourcc-tags.patch
Type: text/x-diff
Size: 1479 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100513/c5683b14/attachment.patch>