[FFmpeg-cvslog] Add PIX_FMT_BGR48LE and PIX_FMT_BGR48BE pixel?formats

[Stefano Sabatini]

On date Saturday 2011-03-19 23:36:22 +0100, Michael Niedermayer wrote:
> On Sat, Mar 19, 2011 at 11:10:57PM +0100, Ivo wrote:
> > On Saturday 19 March 2011, 05:29:01, Peter Ross wrote:
> > > --- a/libavutil/pixfmt.h
> > > +++ b/libavutil/pixfmt.h
> > > @@ -133,6 +133,7 @@ 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_UNUSED,
> > >      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
> > 
> > Isn't it easier to reverse the order of BGR48BE and LE?
> 
> It is before one wrote the patch.
> but once its written its more work :)

$ ffmpeg -pix_fmts
[...]
.O... rgb444be               3            12
.O... rgb444le               3            12
.O... bgr444be               3            12
.O... bgr444le               3            12
I.... y400a                  2            16
..... (null)                 0             0
..... bgr48be                3            48
..... bgr48le                3            48

And I have a bad feeling about this, for example there are many places
where we iterate from 0 to PIX_FMT_NB-1...

-- 
I'm in direct contact with many advanced fun CONCEPTS.