[<<] [<] Page 1 of 1 [>] [>>] | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
Subject:
GrSetSystemPalette
From: George Harvey ####@####.#### Date: 9 Jun 2000 19:35:06 -0000 Message-Id: <yam8195.1571.139376416@smtp.dial.pipex.com> Greg, I think that this may have gone astray when I sent it a few days ago, could you confirm if this patch is OK. Thanks, George > From: George Harvey ####@####.#### > To: ####@####.#### > Date: Sat, 03 Jun 2000 22:02:02 -0000 > Subject: GrSetSystemPalette() > > Hi, > > I am updating my VNC port to use GrSetSystemPalette() and have run > into an inconsistency in the palette setting code in the screen > drivers, specifically the index used to access the palette array. > As far as I can tell, the frame buffer driver uses the correct index > while the X11 driver uses an incorrect index. The change needed to > the X11 driver is only three lines as follows: > > *** scr_x11.cSat Apr 15 03:34:21 2000 > --- /usr/local/src/microwin/src/drivers/scr_x11.cSat Jun 3 22:36:19 2000 > *************** > *** 511,519 **** > XColor exact; > XColor def; > > ! r = pal[first+i].r; > ! g = pal[first+i].g; > ! b = pal[first+i].b; > sprintf(spec, "#%02x%02x%02x", r, g, b); > XLookupColor(x11_dpy, x11_colormap, spec, &exact, &def); > XAllocColor(x11_dpy, x11_colormap, &def); > --- 511,519 ---- > XColor exact; > XColor def; > > ! r = pal[i].r; > ! g = pal[i].g; > ! b = pal[i].b; > sprintf(spec, "#%02x%02x%02x", r, g, b); > XLookupColor(x11_dpy, x11_colormap, spec, &exact, &def); > XAllocColor(x11_dpy, x11_colormap, &def); > > I only have my Prisma driver and the X11 driver to test with > so I just want to confirm that the above change is correct. > Greg, am I on the right track here? > > Thanks, > > George > | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
Subject:
Re: GrSetSystemPalette
From: "Greg Haerr" ####@####.#### Date: 9 Jun 2000 19:41:13 -0000 Message-Id: <05ce01bfd24b$0f07e8e0$3017dbd0@censoft.com> : I think that this may have gone astray when I sent it a few days ago, : could you confirm if this patch is OK. George - I'll try this at home tonight. Hopefully this is the problem that has been plaguing the X11 palette mode! Regards, Greg | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
[<<] [<] Page 1 of 1 [>] [>>] |