[Bug 4140] New: Emu48 with HP49 ROM

[Wine Bugs]

http://bugs.winehq.org/show_bug.cgi?id=4140

Summary: Emu48 with HP49 ROM
           Product: Wine
           Version: unspecified
          Platform: PC-x86-64
        OS/Version: Linux
            Status: UNCONFIRMED
          Severity: normal
          Priority: P2
         Component: wine-misc
        AssignedTo: [e-mail]
        ReportedBy: [e-mail]

Emu48 is a free software (GPL) that can emulate various HP calculators, such as
HP48 and HP49. Source code of emu48 can be downloaded on the maintainer's web site:
http://privat.swol.de/ChristophGiesselink/Emu48/e48sp40.zip

While emulating a HP49 calc, using the HP49 ROM, the screen keeps beeing black,
even if you turn on or reset the emulated calc. This problem has been present
for a while, whatever emu48 version or wine version used (I am using a 32bits
x86 system).
The screen isn't all black with a HP48 ROM, but it is yellow/white instead of
being black/white. One could sometimes get red or blue colors.

To reproduce the bug, please follow these steps (on a 32-bits x86 arch?):
1) Download the package
http://perso.wanadoo.fr/kdntl/misc/winebug_emu48.tar.gz (~800KB) and untar it.
This creates the winebug_emu48 directory with some files in it that I've
gathered in order to ease the manipulation.
2) Create a fresh wine installation.
3) Go into the winebug_emu48/ directory.
4) Run "wine emu48.exe" in a shell. A dialog box is then displayed.
5) In the "Emu48 Directory" text zone, append "\hp49" to the text already
there, to get something like "H:\winebug_emu48\hp49".
Then, click on the down arrow button.
6) Click OK to start the emulation.
7) Click OK to close the report dialog box.
8) Look at the black screen. Click on the calc's ON button. The screen should
still be black instead of displaying something.

I don't know much about the win32 so I didn't manage to patch the original emu48
version. However, there is likely a problem with wine in the display.c file, or
something like that. For example, there is a macro in display.c which is:

#define GRAYMASK(c)     (((((c)-1)>>1)<<24) ^M
                                        |((((c)-1)>>1)<<16) \
                                        |((((c)-1)>>1)<<8)  \
                                        |((((c)-1)>>1)))

Maybe this is not really portable? The maintainer claims that there is no win32
unsupported calls used in emu48.

Thank you in advance for having read my bug report.