From: Bob B. <bb...@us...> - 2003-04-30 16:32:24
|
Could this be an endian-ness issue? On Wed, Apr 30, 2003 at 07:08:21PM +0300, Mario Kadastik <mar...@mi...> wrote: > Ok, I think I solved the problem, but that's for solaris only and is > only a temporary > fix until someone implements it a bit more nicely with check for solaris. > > Copy lines after my name to xwin.patch and run: > patch xwin.c < xwin.patch > > All I actually did was to replace red and blue in split_colour24 and > translate24to32 > > Mario > > 183c183 > < rv.blue = (colour & 0xff0000) >> 16; > --- > > rv.red = (colour & 0xff0000) >> 16; > 185c185 > < rv.red = (colour & 0xff); > --- > > rv.blue = (colour & 0xff); > 404c404 > < pixel = *(data++); > --- > > pixel = *(data++) << 16; > 406c406 > < pixel |= *(data++) << 16; > --- > > pixel |= *(data++); > > > Mario Kadastik wrote: > > >More debugging: > > > >Col1: 6a240a Col2: a246a > >Col1: 11010100010010000001010 > >Col2: 00010100010010001101010 > > > >Col1: c8d0d4 Col2: d4d0c8 > >Col1: 110010001101000011010100 > >Col2: 110101001101000011001000 > > > >So it seems that for some odd reason red and blue are swapped. > >col1 is before make_colour32(split_colour24(xx)) and col2 is after. > > > >Mario > > > > > > ------------------------------------------------------- > This sf.net email is sponsored by:ThinkGeek > Welcome to geek heaven. > http://thinkgeek.com/sf > _______________________________________________ > rdesktop-users mailing list > rde...@li... > https://lists.sourceforge.net/lists/listinfo/rdesktop-users -- Bob Bell <bb...@us...> |