Q's patch and my adaptation for tcx just reversed the order of the
colors, just like BlueSwirl's patch except that they used #if
__LITTLE_ENDIAN__ in place of #ifdef WORDS_BIGENDIAN, and this
worked
for both i386 and sparc32 guests on i386 host (but is not applied
for
ppc host).
http://www.kju-app.org/proj/browser/trunk/patches/q_vga.c_02.diff
I think this is not correct either, instead the DisplayState bgr
attribute should be used. This version should work like before,
and on
BGR displays the colours should be correct. VGA needs a similar
change
for 15 and 16 bit depths if I'm correct.