# HG changeset patch # User Sam Lantinga # Date 1233297524 0 # Node ID b7197d7e856688ed1ff7e4dbeaeacf02192214ce # Parent 1a45bc89db0c21db5b54915d2cebdb28cb64c70f indent diff -r 1a45bc89db0c -r b7197d7e8566 src/video/x11/SDL_x11gamma.c --- a/src/video/x11/SDL_x11gamma.c Thu Jan 29 13:25:08 2009 +0000 +++ b/src/video/x11/SDL_x11gamma.c Fri Jan 30 06:38:44 2009 +0000 @@ -97,21 +97,21 @@ cmapTable[numCmaps].ramp = NULL; if (ramp != NULL) { - newramp = SDL_malloc(SDL_GammaRampSize); - if (NULL == newramp) { - SDL_SetError("Out of memory in X11_TrackColormap()"); - return; - } - SDL_memset(newramp, 0, SDL_GammaRampSize); - cmapTable[numCmaps].ramp = newramp; + newramp = SDL_malloc(SDL_GammaRampSize); + if (NULL == newramp) { + SDL_SetError("Out of memory in X11_TrackColormap()"); + return; + } + SDL_memset(newramp, 0, SDL_GammaRampSize); + cmapTable[numCmaps].ramp = newramp; - ncolors = cmapTable[numCmaps].visual.map_entries; + ncolors = cmapTable[numCmaps].visual.map_entries; - for (i = 0; i < ncolors; i++) { - newramp[(0 * 256) + i] = ramp[i].red; - newramp[(1 * 256) + i] = ramp[i].green; - newramp[(2 * 256) + i] = ramp[i].blue; - } + for (i = 0; i < ncolors; i++) { + newramp[(0 * 256) + i] = ramp[i].red; + newramp[(1 * 256) + i] = ramp[i].green; + newramp[(2 * 256) + i] = ramp[i].blue; + } } numCmaps++; @@ -149,14 +149,14 @@ return -1; } /* remember the new ramp */ - if (cmapTable[j].ramp == NULL) { - Uint16 * newramp = SDL_malloc(SDL_GammaRampSize); - if (NULL == newramp) { - SDL_SetError("Out of memory in X11_TrackColormap()"); - return -1; - } - cmapTable[j].ramp = newramp; - } + if (cmapTable[j].ramp == NULL) { + Uint16 *newramp = SDL_malloc(SDL_GammaRampSize); + if (NULL == newramp) { + SDL_SetError("Out of memory in X11_TrackColormap()"); + return -1; + } + cmapTable[j].ramp = newramp; + } SDL_memcpy(cmapTable[j].ramp, ramp, SDL_GammaRampSize); rshift = 0; diff -r 1a45bc89db0c -r b7197d7e8566 src/video/x11/SDL_x11window.c --- a/src/video/x11/SDL_x11window.c Thu Jan 29 13:25:08 2009 +0000 +++ b/src/video/x11/SDL_x11window.c Fri Jan 30 06:38:44 2009 +0000 @@ -220,14 +220,14 @@ XStandardColormap cmap; XColor *colorcells; Colormap colormap; - Sint32 pix; + Sint32 pix; Sint32 ncolors; Sint32 nbits; Sint32 rmax, gmax, bmax; - Sint32 rwidth, gwidth, bwidth; + Sint32 rwidth, gwidth, bwidth; Sint32 rmask, gmask, bmask; Sint32 rshift, gshift, bshift; - Sint32 r, g, b; + Sint32 r, g, b; /* Is the colormap we need already registered in SDL? */ if (colormap = @@ -259,26 +259,26 @@ } ncolors = visual->map_entries; - nbits = visual->bits_per_rgb; + nbits = visual->bits_per_rgb; /* printf("ncolors = %d nbits = %d\n", ncolors, nbits); */ - /* what if ncolors != (1 << nbits)? That can happen on a - true PseudoColor display. I'm assuming that we will - always have ncolors == (1 << nbits)*/ + /* what if ncolors != (1 << nbits)? That can happen on a + true PseudoColor display. I'm assuming that we will + always have ncolors == (1 << nbits) */ + + /* I'm making a lot of assumptions here. */ - /* I'm making a lot of assumptions here. */ - - /* Compute the width of each field. If there is one extra - bit, give it to green. If there are two extra bits give - them to red and greed. We can get extra bits when the - number of bits per pixel is not a multiple of 3. For - example when we have 16 bits per pixel and need a 5/6/5 - layout for the RGB fields */ + /* Compute the width of each field. If there is one extra + bit, give it to green. If there are two extra bits give + them to red and greed. We can get extra bits when the + number of bits per pixel is not a multiple of 3. For + example when we have 16 bits per pixel and need a 5/6/5 + layout for the RGB fields */ - rwidth = (nbits / 3) + (((nbits % 3) == 2) ? 1 : 0); - gwidth = (nbits / 3) + (((nbits % 3) >= 1) ? 1 : 0); - bwidth = (nbits / 3); + rwidth = (nbits / 3) + (((nbits % 3) == 2) ? 1 : 0); + gwidth = (nbits / 3) + (((nbits % 3) >= 1) ? 1 : 0); + bwidth = (nbits / 3); rshift = gwidth + bwidth; gshift = bwidth; @@ -297,24 +297,25 @@ /* printf("blue mask = %4x shift = %4d width = %d\n", bmask, bshift, bwidth); */ /* build the color table pixel values */ - pix = 0; - for (r = 0; r < rmax; r++) { - for (g = 0; g < gmax; g++) { - for (b = 0; b < bmax; b++) { - colorcells[pix].pixel = (r << rshift) | (g << gshift) | (b << bshift); - colorcells[pix].red = (0xffff * r) / rmask; - colorcells[pix].green = (0xffff * g) / gmask; - colorcells[pix].blue = (0xffff * b) / bmask; + pix = 0; + for (r = 0; r < rmax; r++) { + for (g = 0; g < gmax; g++) { + for (b = 0; b < bmax; b++) { + colorcells[pix].pixel = + (r << rshift) | (g << gshift) | (b << bshift); + colorcells[pix].red = (0xffff * r) / rmask; + colorcells[pix].green = (0xffff * g) / gmask; + colorcells[pix].blue = (0xffff * b) / bmask; /* printf("%4x:%4x [%4x %4x %4x]\n", */ /* pix, */ /* colorcells[pix].pixel, */ /* colorcells[pix].red, */ /* colorcells[pix].green, */ /* colorcells[pix].blue); */ - pix++; - } - } - } + pix++; + } + } + } /* status = */ /* XStoreColors(data->display, colormap, colorcells, ncolors); */