[Ffmpeg-devel] [PATCH] adds missing rgb/bgr 15 tags to raw decoder

Michael Niedermayer michaelni
Fri Oct 27 18:29:44 CEST 2006


Hi

On Fri, Oct 27, 2006 at 12:48:06PM -0300, Reynaldo H. Verdejo Pinochet wrote:
> On Fri, Oct 27, 2006 at 05:36:25PM +0200, Diego Biurrun wrote:
> > >      { PIX_FMT_GRAY8,   MKTAG('G', 'R', 'E', 'Y') },
> > > +    { PIX_FMT_RGB555,   MKTAG('R', 'G', 'B', 15) },
> > > +    { PIX_FMT_BGR555,   MKTAG('B', 'G', 'R', 15) },
> > 
> > nit: Please align the MKTAG when you commit this.
> > 
> > Diego
> 
> Damn, 6 errors in two lines.. this should be a record. sorry
> for the time I've made you waste, fixed on my three.

patch ok _after_ you double check that the RGB vs. BGR thing matches
mplayer (aka mplayers raw decoder and liavcodecs raw.c produce correct
red and blue colors

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

In the past you could go to a library and read, borrow or copy any book
Today you'd get arrested for mere telling someone where the library is




More information about the ffmpeg-devel mailing list