[FFmpeg-devel] [PATCH] Fix swapped colours for RGB555 and RGB565 in swscale
Mon Oct 29 15:40:08 CET 2007
On Sun, Oct 28, 2007 at 12:12:08AM +0200, Michael Niedermayer wrote:
> On Wed, Oct 24, 2007 at 03:48:00PM +0300, Kostya wrote:
> > Looks like rgb1toUV have color values swapped.
> > Fixes roundup issue 162 and reported problem with 15-bit TSCC files.
> this patch is TOTALLY wrong
> it changes rgb1toUV so they become identical to bgr1toUV and at the
> same time results in the corresponding luma functions to mismatch
> which means that after this the output is _really_ wrong while before it
> was merely switching red and blue
> the actual bug seems to be that the function names of [rgb|bgr]1to[UV|Y]
> should be rgb<->bgr flipped
> i also suspect that mplayers vf_scale.c might need a change to compensate for
You are right, MPlayer interprets rgb1 as bgr1 and vice
versa, so other rgb1 converters are wrong too. I don't know
what consequences name fix will have as other apps have grown on
that model too.
> Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
More information about the ffmpeg-devel