On Sun, Mar 04, 2001 at 01:51:16PM +0100, Ove Kaaven wrote:
Since this is a 32-bit move (movl), you should use %%eax, not %%ax. The assembler doesn't complain about this now, but technically it should, and might someday.
Since you're converting to 32bpp, not 16bpp, you should multiply "left" by 4, not by 2, or strange things may happen in some apps.
Thanks. Fixed patch attached. I did change left multiplicator from 2 to 4 in my earlier patches, but I forgot to include it in my final patch being 3 AM and so... I'll try to be more careful next time.
Vedran