[FFmpeg-devel] [PATCH] Make 15-bpp MS Video 1 decoder not output 16-bpp

[Michael Niedermayer]

On Mon, Mar 09, 2009 at 07:20:11PM +0200, Kostya wrote:
> On Mon, Mar 09, 2009 at 04:14:47PM +0100, Michael Niedermayer wrote:
> > On Mon, Mar 09, 2009 at 08:41:35AM +0200, Kostya wrote:
> > > On Sun, Mar 08, 2009 at 10:13:54PM +0100, Michael Niedermayer wrote:
> > > > On Sun, Mar 08, 2009 at 11:31:49PM +0200, Kostya wrote:
> > > > > 15-bpp MS Video 1 decoder first colour high bit is used
> > > > > to indicate coding mode (2 or 8-colour fill). Passing
> > > > > that value as such to current swscaler disrupt bit magic
> > > > > on conversion and you can see 2x2 red rectangles in
> > > > > different places.
> > > > > 
> > > > > Some samples like
> > > > > http://samples.mplayerhq.hu/V-codecs/CRAM/orbean.avi
> > > > > manifest that problem.
> > > > > 
> > > > 
> > > > > Attached SwScaler patch for reference purpose only.
> > > > 
> > > > the bug is that maska is not set to the remaining bit
> > > 
> > > How's this?
> > 
> > ok if tested
> 
> It is, but how about this patch (also works fine)?
> I feel it is a bit more correct.

fine as well until someone decides the extra bit is alpha ...

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

The worst form of inequality is to try to make unequal things equal.
-- Aristotle
-------------- 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/20090309/3b338429/attachment.pgp>