[FFmpeg-devel] [PATCH] Misc patches for pixdesc.c

Michael Niedermayer michaelni
Tue Mar 31 05:05:01 CEST 2009


On Mon, Mar 30, 2009 at 01:12:51AM +0200, Stefano Sabatini wrote:
> On date Sunday 2009-03-29 23:18:44 +0200, Michael Niedermayer encoded:
> > On Tue, Mar 24, 2009 at 09:51:26PM +0100, Stefano Sabatini wrote:
> > > Hi all,
> > > 
> > > cosmetics+reorder patch.
> > > 
> > > Second patch is unfortunately unreadable, what it does is to resort
> > > the pixdescs in the same order as the pixfmts, that should ease my
> > > future work of filling the gaps.
> > [...]
> > > @@ -148,9 +148,9 @@
> > >          .log2_chroma_w= 0,
> > >          .log2_chroma_h= 0,
> > >          .comp = {
> > > -            {0,5,1,0,15},
> > > -            {0,5,3,0,15},
> > > -            {0,5,5,0,15},
> > > +            {0,5,1,0,15},       /* R */
> > > +            {0,5,3,0,15},       /* G */
> > > +            {0,5,5,0,15},       /* B */
> > >          },
> > >          .flags = PIX_FMT_BE,
> > >      },
> > > @@ -160,9 +160,9 @@
> > >          .log2_chroma_w= 0,
> > >          .log2_chroma_h= 0,
> > >          .comp = {
> > > -            {0,1,1,0,4},
> > > -            {0,1,1,5,5},
> > > -            {0,1,2,3,4},
> > > +            {0,1,2,3,4},        /* R */
> > > +            {0,1,1,5,5},        /* G */
> > > +            {0,1,1,0,4},        /* B */
> > >          },
> > >      },
> > >      [PIX_FMT_RGB565BE] = {
> > 
> > mix of cosmetic and functional changes
> 
> Split and updated.

adding comments and reorder is ok if they do nothing else
(check obj files to match ...)

RGB<->BGR is ok too

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Freedom in capitalist society always remains about the same as it was in
ancient Greek republics: Freedom for slave owners. -- Vladimir Lenin
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090331/ad4f2ae5/attachment.pgp>



More information about the ffmpeg-devel mailing list