Blue Swirl wrote: > On 6/8/07, Thiemo Seufer <[EMAIL PROTECTED]> wrote: > >Blue Swirl wrote: > >[snip] > >> >I haven't been able to test the new patch yet. > >> > > >> >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. > > > >This one doesn't change anything for my testcases, so I guess it > >is ok for me. > > Could you test if this version fixes the 16/15 bit PPC host case?
Again, no change for me. Thiemo