http://bugs.winehq.org/show_bug.cgi?id=11203
--- Comment #24 from Ian Goddard igoddard@nildram.co.uk 2008-09-29 17:38:29 --- I have the same problem on both my laptop and a Via thin client. I now take each release from git, build it and run the test suite (igoddard_laptop) and check that the problem still exists with EA. I then make a one line change and install it (no more fontforge problems nowadays!). I ran the test suite on one of these rebuilds. It fixed a whole bunch of test failures.
On the face of it the code could be greatly simplified by reducing the switch at about line 300 of x11drv_main.c to return 16 on case 15, return depth on all the other existing cases and leave the default in place.
However, the comment about opengl is a worry - if this were done would it break opengl apps which could use a 32-bit depth from some H/W which reports 24? This is why I haven't submitted a patch. If the test suite tests for such opengl cases maybe it would be possible to put some such fix into a dev release and be prepared to back it out with the intent of putting something better in place if it does break tests.
Ian