msvidc: Fix 16bit VIDEO-1/CRAM decompression.

Joerg-Cyril.Hoehle at t-systems.com Joerg-Cyril.Hoehle at t-systems.com
Mon Apr 12 05:16:21 CDT 2010


Hi,

This patch allows all videos in "Ritter Rost - die eiserne Burg"
to be rendered correctly (with the help of native mciavi).
I have yet to write an AppDB entry for that app (silver rating with patch).

Prior to that, the videos were rendered upside down and with a sort
of interleave such that only the upper half was visible, and the bottom
half likely overwriting memory somewhere.

Ritter Rost does not use a negative height in the BITMAPINFOHEADER.

The patch clearly shows what sections of code might be changed to
support top-down bitmaps with negative height that Maarten Lankhorst
mentioned recently.  That's a topic for a future patch, as I'm not
familiar with these (and not aware of any programs using them with this codec).

The two decoder functions are identical to what's currently in the FFmpeg
code tree, except for the tiny #ifdef ORIGINAL section.

But why is there a need to invert the orientation against the original FFmpeg code?

Regards,
 Jörg Höhle
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-msvidc-Fix-16bit-CRAM-decompression.patch
Type: application/octet-stream
Size: 5561 bytes
Desc: 0001-msvidc-Fix-16bit-CRAM-decompression.patch
URL: <http://www.winehq.org/pipermail/wine-patches/attachments/20100412/25c3a148/attachment-0001.obj>


More information about the wine-patches mailing list