[FFmpeg-cvslog] Add PIX_FMT_BGR48LE and PIX_FMT_BGR48BE pixel formats

[Michael Niedermayer]

On Sat, Mar 19, 2011 at 03:29:01PM +1100, Peter Ross wrote:
> On Thu, Mar 17, 2011 at 03:52:47PM +0100, Peter Ross wrote:
> > ffmpeg | branch: master | Peter Ross <pross at xvid.org> | Thu Mar 17 21:15:57 2011 +1100| [e309fdc7018a1027d187ec27fb1d69a41a4ee167] | committer: Michael Niedermayer
> > 
> > Add PIX_FMT_BGR48LE and PIX_FMT_BGR48BE pixel formats
> > 
> [..]
> 
> > --- a/libavutil/pixfmt.h
> > +++ b/libavutil/pixfmt.h
> > @@ -133,6 +133,8 @@ enum PixelFormat {
> >      PIX_FMT_BGR444BE,  ///< packed BGR 4:4:4, 16bpp, (msb)4A 4B 4G 4R(lsb), big-endian, most significant bits to 1
> >      PIX_FMT_BGR444LE,  ///< packed BGR 4:4:4, 16bpp, (msb)4A 4B 4G 4R(lsb), little-endian, most significant bits to 1
> >      PIX_FMT_Y400A,     ///< 8bit gray, 8bit alpha
> > +    PIX_FMT_BGR48BE,   ///< packed RGB 16:16:16, 48bpp, 16B, 16G, 16R, the 2-byte value for each R/G/B component is stored as big-endian
> > +    PIX_FMT_BGR48LE,   ///< packed RGB 16:16:16, 48bpp, 16B, 16G, 16R, the 2-byte value for each R/G/B component is stored as little-endian
> >      PIX_FMT_NB,        ///< number of pixel formats, DO NOT USE THIS if you want to link with shared libav* because the number of formats might differ between versions
> >  };
> 
> Problem: libswscale's isBE(pixfmt) macro examines the least significant bit of the pixel
> format value to determine pixel format endianess. Odd is big-endian, even is little endian.
> 
> PIX_FMT_Y400A interrupts the order of the enums, so the PIX_FMT_BGR48BE and PIX_FMT_BGR48LE
> values are wrong. Fix enclosed.

thx, applied

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

He who knows, does not speak. He who speaks, does not know. -- Lao Tsu
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-cvslog/attachments/20110319/c93eb9c4/attachment-0001.asc>