1 files changed, 370 insertions, 439 deletions

diff --git a/hw/xwin/wincmap.c b/hw/xwin/wincmap.c
index d526a9201..610437ad4 100644
--- a/hw/xwin/wincmap.c
+++ b/hw/xwin/wincmap.c
@@ -36,59 +36,56 @@
 #endif
 #include "win.h"
 
-
 /*
  * Local prototypes
  */
 
 static int
-winListInstalledColormaps (ScreenPtr pScreen, Colormap *pmaps);
+ winListInstalledColormaps(ScreenPtr pScreen, Colormap * pmaps);
 
 static void
-winStoreColors (ColormapPtr pmap, int ndef, xColorItem *pdefs);
+ winStoreColors(ColormapPtr pmap, int ndef, xColorItem * pdefs);
 
 static void
-winInstallColormap (ColormapPtr pmap);
+ winInstallColormap(ColormapPtr pmap);
 
 static void
-winUninstallColormap (ColormapPtr pmap);
+ winUninstallColormap(ColormapPtr pmap);
 
 static void
-winResolveColor (unsigned short *pred,
-		 unsigned short *pgreen,
-		 unsigned short *pblue,
-		 VisualPtr	pVisual);
+
+winResolveColor(unsigned short *pred,
+                unsigned short *pgreen,
+                unsigned short *pblue, VisualPtr pVisual);
 
 static Bool
-winCreateColormap (ColormapPtr pmap);
+ winCreateColormap(ColormapPtr pmap);
 
 static void
-winDestroyColormap (ColormapPtr pmap);
+ winDestroyColormap(ColormapPtr pmap);
 
 static Bool
-winGetPaletteDIB (ScreenPtr pScreen, ColormapPtr pcmap);
+ winGetPaletteDIB(ScreenPtr pScreen, ColormapPtr pcmap);
 
 static Bool
-winGetPaletteDD (ScreenPtr pScreen, ColormapPtr pcmap);
-
+ winGetPaletteDD(ScreenPtr pScreen, ColormapPtr pcmap);
 
 /*
  * Set screen functions for colormaps
  */
 
 void
-winSetColormapFunctions (ScreenPtr pScreen)
+winSetColormapFunctions(ScreenPtr pScreen)
 {
-  pScreen->CreateColormap = winCreateColormap;
-  pScreen->DestroyColormap = winDestroyColormap;
-  pScreen->InstallColormap = winInstallColormap;
-  pScreen->UninstallColormap = winUninstallColormap;
-  pScreen->ListInstalledColormaps = winListInstalledColormaps;
-  pScreen->StoreColors = winStoreColors;
-  pScreen->ResolveColor = winResolveColor;
+    pScreen->CreateColormap = winCreateColormap;
+    pScreen->DestroyColormap = winDestroyColormap;
+    pScreen->InstallColormap = winInstallColormap;
+    pScreen->UninstallColormap = winUninstallColormap;
+    pScreen->ListInstalledColormaps = winListInstalledColormaps;
+    pScreen->StoreColors = winStoreColors;
+    pScreen->ResolveColor = winResolveColor;
 }
 
-
 /* See Porting Layer Definition - p. 30 */
 /*
  * Walk the list of installed colormaps, filling the pmaps list
@@ -96,576 +93,510 @@ winSetColormapFunctions (ScreenPtr pScreen)
  * a count of the total number of installed maps.
  */
 static int
-winListInstalledColormaps (ScreenPtr pScreen, Colormap *pmaps)
+winListInstalledColormaps(ScreenPtr pScreen, Colormap * pmaps)
 {
-  winScreenPriv(pScreen);
-
-  /*
-   * There will only be one installed colormap, so we only need
-   * to return one id, and the count of installed maps will always
-   * be one.
-   */
-  *pmaps = pScreenPriv->pcmapInstalled->mid;
-  return 1;
+    winScreenPriv(pScreen);
+
+    /*
+     * There will only be one installed colormap, so we only need
+     * to return one id, and the count of installed maps will always
+     * be one.
+     */
+    *pmaps = pScreenPriv->pcmapInstalled->mid;
+    return 1;
 }
 
-
 /* See Porting Layer Definition - p. 30 */
 /* See Programming Windows - p. 663 */
 static void
-winInstallColormap (ColormapPtr pColormap)
+winInstallColormap(ColormapPtr pColormap)
 {
-  ScreenPtr		pScreen = pColormap->pScreen;
-  winScreenPriv(pScreen);
-  ColormapPtr		oldpmap = pScreenPriv->pcmapInstalled;
+    ScreenPtr pScreen = pColormap->pScreen;
+
+    winScreenPriv(pScreen);
+    ColormapPtr oldpmap = pScreenPriv->pcmapInstalled;
 
 #if CYGDEBUG
-  winDebug ("winInstallColormap\n");
+    winDebug("winInstallColormap\n");
 #endif
- 
-  /* Did the colormap actually change? */
-  if (pColormap != oldpmap)
-    {
+
+    /* Did the colormap actually change? */
+    if (pColormap != oldpmap) {
 #if CYGDEBUG
-      winDebug ("winInstallColormap - Colormap has changed, attempt "
-	      "to install.\n");
+        winDebug("winInstallColormap - Colormap has changed, attempt "
+                 "to install.\n");
 #endif
-      
-      /* Was there a previous colormap? */
-      if (oldpmap != (ColormapPtr) None)
-	{
-	  /* There was a previous colormap; tell clients it is gone */
-	  WalkTree (pColormap->pScreen, TellLostMap, (char *)&oldpmap->mid);
-	}
-      
-      /* Install new colormap */
-      pScreenPriv->pcmapInstalled = pColormap;
-      WalkTree (pColormap->pScreen, TellGainedMap, (char *)&pColormap->mid);
-      
-      /* Call the engine specific colormap install procedure */
-      if (!((*pScreenPriv->pwinInstallColormap) (pColormap)))
-	{
-	  winErrorFVerb (2, "winInstallColormap - Screen specific colormap install "
-		  "procedure failed.  Continuing, but colors may be "
-		  "messed up from now on.\n");
-	}
+
+        /* Was there a previous colormap? */
+        if (oldpmap != (ColormapPtr) None) {
+            /* There was a previous colormap; tell clients it is gone */
+            WalkTree(pColormap->pScreen, TellLostMap, (char *) &oldpmap->mid);
+        }
+
+        /* Install new colormap */
+        pScreenPriv->pcmapInstalled = pColormap;
+        WalkTree(pColormap->pScreen, TellGainedMap, (char *) &pColormap->mid);
+
+        /* Call the engine specific colormap install procedure */
+        if (!((*pScreenPriv->pwinInstallColormap) (pColormap))) {
+            winErrorFVerb(2,
+                          "winInstallColormap - Screen specific colormap install "
+                          "procedure failed.  Continuing, but colors may be "
+                          "messed up from now on.\n");
+        }
     }
 
-  /* Save a pointer to the newly installed colormap */
-  pScreenPriv->pcmapInstalled = pColormap;
+    /* Save a pointer to the newly installed colormap */
+    pScreenPriv->pcmapInstalled = pColormap;
 }
 
-
 /* See Porting Layer Definition - p. 30 */
 static void
-winUninstallColormap (ColormapPtr pmap)
+winUninstallColormap(ColormapPtr pmap)
 {
-  winScreenPriv(pmap->pScreen);
-  ColormapPtr curpmap = pScreenPriv->pcmapInstalled;
+    winScreenPriv(pmap->pScreen);
+    ColormapPtr curpmap = pScreenPriv->pcmapInstalled;
 
 #if CYGDEBUG
-  winDebug ("winUninstallColormap\n");
+    winDebug("winUninstallColormap\n");
 #endif
 
-  /* Is the colormap currently installed? */
-  if (pmap != curpmap)
-    {
-      /* Colormap not installed, nothing to do */
-      return;
+    /* Is the colormap currently installed? */
+    if (pmap != curpmap) {
+        /* Colormap not installed, nothing to do */
+        return;
     }
-  
-  /* Clear the installed colormap flag */
-  pScreenPriv->pcmapInstalled = NULL;
-  
-  /*
-   * NOTE: The default colormap does not get "uninstalled" before
-   * it is destroyed.
-   */
-
-  /* Install the default cmap in place of the cmap to be uninstalled */
-  if (pmap->mid != pmap->pScreen->defColormap)
-    {
-      dixLookupResourceByType((pointer) &curpmap, pmap->pScreen->defColormap,
-				RT_COLORMAP, NullClient, DixUnknownAccess);
-      (*pmap->pScreen->InstallColormap) (curpmap);
+
+    /* Clear the installed colormap flag */
+    pScreenPriv->pcmapInstalled = NULL;
+
+    /*
+     * NOTE: The default colormap does not get "uninstalled" before
+     * it is destroyed.
+     */
+
+    /* Install the default cmap in place of the cmap to be uninstalled */
+    if (pmap->mid != pmap->pScreen->defColormap) {
+        dixLookupResourceByType((pointer) &curpmap, pmap->pScreen->defColormap,
+                                RT_COLORMAP, NullClient, DixUnknownAccess);
+        (*pmap->pScreen->InstallColormap) (curpmap);
     }
 }
 
-
 /* See Porting Layer Definition - p. 30 */
 static void
-winStoreColors (ColormapPtr pmap,
-		int ndef,
-		xColorItem *pdefs)
+winStoreColors(ColormapPtr pmap, int ndef, xColorItem * pdefs)
 {
-  ScreenPtr		pScreen = pmap->pScreen;
-  winScreenPriv(pScreen);
-  winCmapPriv(pmap);
-  int			i;
-  unsigned short	nRed, nGreen, nBlue;
+    ScreenPtr pScreen = pmap->pScreen;
+
+    winScreenPriv(pScreen);
+    winCmapPriv(pmap);
+    int i;
+    unsigned short nRed, nGreen, nBlue;
 
 #if CYGDEBUG
-  if (ndef != 1)
-    winDebug ("winStoreColors - ndef: %d\n",
-	    ndef);
+    if (ndef != 1)
+        winDebug("winStoreColors - ndef: %d\n", ndef);
 #endif
 
-  /* Save the new colors in the colormap privates */
-  for (i = 0; i < ndef; ++i)
-    {
-      /* Adjust the colors from the X color spec to the Windows color spec */
-      nRed = pdefs[i].red >> 8;
-      nGreen = pdefs[i].green >> 8;
-      nBlue = pdefs[i].blue >> 8;
-
-      /* Copy the colors to a palette entry table */
-      pCmapPriv->peColors[pdefs[0].pixel + i].peRed = nRed;
-      pCmapPriv->peColors[pdefs[0].pixel + i].peGreen = nGreen;
-      pCmapPriv->peColors[pdefs[0].pixel + i].peBlue = nBlue;
-      
-      /* Copy the colors to a RGBQUAD table */
-      pCmapPriv->rgbColors[pdefs[0].pixel + i].rgbRed = nRed;
-      pCmapPriv->rgbColors[pdefs[0].pixel + i].rgbGreen = nGreen;
-      pCmapPriv->rgbColors[pdefs[0].pixel + i].rgbBlue = nBlue;
+    /* Save the new colors in the colormap privates */
+    for (i = 0; i < ndef; ++i) {
+        /* Adjust the colors from the X color spec to the Windows color spec */
+        nRed = pdefs[i].red >> 8;
+        nGreen = pdefs[i].green >> 8;
+        nBlue = pdefs[i].blue >> 8;
+
+        /* Copy the colors to a palette entry table */
+        pCmapPriv->peColors[pdefs[0].pixel + i].peRed = nRed;
+        pCmapPriv->peColors[pdefs[0].pixel + i].peGreen = nGreen;
+        pCmapPriv->peColors[pdefs[0].pixel + i].peBlue = nBlue;
+
+        /* Copy the colors to a RGBQUAD table */
+        pCmapPriv->rgbColors[pdefs[0].pixel + i].rgbRed = nRed;
+        pCmapPriv->rgbColors[pdefs[0].pixel + i].rgbGreen = nGreen;
+        pCmapPriv->rgbColors[pdefs[0].pixel + i].rgbBlue = nBlue;
 
 #if CYGDEBUG
-      winDebug ("winStoreColors - nRed %d nGreen %d nBlue %d\n",
-	      nRed, nGreen, nBlue);
+        winDebug("winStoreColors - nRed %d nGreen %d nBlue %d\n",
+                 nRed, nGreen, nBlue);
 #endif
     }
 
-  /* Call the engine specific store colors procedure */
-  if (!((pScreenPriv->pwinStoreColors) (pmap, ndef, pdefs)))
-    {
-      winErrorFVerb (2, "winStoreColors - Engine cpecific color storage procedure "
-	      "failed.  Continuing, but colors may be messed up from now "
-	      "on.\n");
+    /* Call the engine specific store colors procedure */
+    if (!((pScreenPriv->pwinStoreColors) (pmap, ndef, pdefs))) {
+        winErrorFVerb(2,
+                      "winStoreColors - Engine cpecific color storage procedure "
+                      "failed.  Continuing, but colors may be messed up from now "
+                      "on.\n");
     }
 }
 
-
 /* See Porting Layer Definition - p. 30 */
 static void
-winResolveColor (unsigned short *pred,
-		 unsigned short *pgreen,
-		 unsigned short *pblue,
-		 VisualPtr	pVisual)
+winResolveColor(unsigned short *pred,
+                unsigned short *pgreen,
+                unsigned short *pblue, VisualPtr pVisual)
 {
 #if CYGDEBUG
-  winDebug ("winResolveColor ()\n");
+    winDebug("winResolveColor ()\n");
 #endif
 
-  miResolveColor (pred, pgreen, pblue, pVisual);
+    miResolveColor(pred, pgreen, pblue, pVisual);
 }
 
-
 /* See Porting Layer Definition - p. 29 */
 static Bool
-winCreateColormap (ColormapPtr pmap)
+winCreateColormap(ColormapPtr pmap)
 {
-  winPrivCmapPtr	pCmapPriv = NULL;
-  ScreenPtr		pScreen = pmap->pScreen;
-  winScreenPriv(pScreen);
+    winPrivCmapPtr pCmapPriv = NULL;
+    ScreenPtr pScreen = pmap->pScreen;
+
+    winScreenPriv(pScreen);
 
 #if CYGDEBUG
-  winDebug ("winCreateColormap\n");
+    winDebug("winCreateColormap\n");
 #endif
 
-  /* Allocate colormap privates */
-  if (!winAllocateCmapPrivates (pmap))
-    {
-      ErrorF ("winCreateColorma - Couldn't allocate cmap privates\n");
-      return FALSE;
+    /* Allocate colormap privates */
+    if (!winAllocateCmapPrivates(pmap)) {
+        ErrorF("winCreateColorma - Couldn't allocate cmap privates\n");
+        return FALSE;
     }
 
-  /* Get a pointer to the newly allocated privates */
-  pCmapPriv = winGetCmapPriv (pmap);
-
-  /*
-   * FIXME: This is some evil hackery to help in handling some X clients
-   * that expect the top pixel to be white.  This "help" only lasts until
-   * some client overwrites the top colormap entry.
-   * 
-   * We don't want to actually allocate the top entry, as that causes
-   * problems with X clients that need 7 planes (128 colors) in the default
-   * colormap, such as Magic 7.1.
-   */
-  pCmapPriv->rgbColors[WIN_NUM_PALETTE_ENTRIES - 1].rgbRed = 255;
-  pCmapPriv->rgbColors[WIN_NUM_PALETTE_ENTRIES - 1].rgbGreen = 255;
-  pCmapPriv->rgbColors[WIN_NUM_PALETTE_ENTRIES - 1].rgbBlue = 255;
-  pCmapPriv->peColors[WIN_NUM_PALETTE_ENTRIES - 1].peRed = 255;
-  pCmapPriv->peColors[WIN_NUM_PALETTE_ENTRIES - 1].peGreen = 255;
-  pCmapPriv->peColors[WIN_NUM_PALETTE_ENTRIES - 1].peBlue = 255;
-
-  /* Call the engine specific colormap initialization procedure */
-  if (!((*pScreenPriv->pwinCreateColormap) (pmap)))
-    {
-      ErrorF ("winCreateColormap - Engine specific colormap creation "
-	      "procedure failed.  Aborting.\n");
-      return FALSE;
+    /* Get a pointer to the newly allocated privates */
+    pCmapPriv = winGetCmapPriv(pmap);
+
+    /*
+     * FIXME: This is some evil hackery to help in handling some X clients
+     * that expect the top pixel to be white.  This "help" only lasts until
+     * some client overwrites the top colormap entry.
+     * 
+     * We don't want to actually allocate the top entry, as that causes
+     * problems with X clients that need 7 planes (128 colors) in the default
+     * colormap, such as Magic 7.1.
+     */
+    pCmapPriv->rgbColors[WIN_NUM_PALETTE_ENTRIES - 1].rgbRed = 255;
+    pCmapPriv->rgbColors[WIN_NUM_PALETTE_ENTRIES - 1].rgbGreen = 255;
+    pCmapPriv->rgbColors[WIN_NUM_PALETTE_ENTRIES - 1].rgbBlue = 255;
+    pCmapPriv->peColors[WIN_NUM_PALETTE_ENTRIES - 1].peRed = 255;
+    pCmapPriv->peColors[WIN_NUM_PALETTE_ENTRIES - 1].peGreen = 255;
+    pCmapPriv->peColors[WIN_NUM_PALETTE_ENTRIES - 1].peBlue = 255;
+
+    /* Call the engine specific colormap initialization procedure */
+    if (!((*pScreenPriv->pwinCreateColormap) (pmap))) {
+        ErrorF("winCreateColormap - Engine specific colormap creation "
+               "procedure failed.  Aborting.\n");
+        return FALSE;
     }
 
-  return TRUE;
+    return TRUE;
 }
 
-
 /* See Porting Layer Definition - p. 29, 30 */
 static void
-winDestroyColormap (ColormapPtr pColormap)
+winDestroyColormap(ColormapPtr pColormap)
 {
-  winScreenPriv(pColormap->pScreen);
-  winCmapPriv(pColormap);
-
-  /* Call the engine specific colormap destruction procedure */
-  if (!((*pScreenPriv->pwinDestroyColormap) (pColormap)))
-    {
-      winErrorFVerb (2, "winDestroyColormap - Engine specific colormap destruction "
-	      "procedure failed.  Continuing, but it is possible that memory "
-	      "was leaked, or that colors will be messed up from now on.\n");
+    winScreenPriv(pColormap->pScreen);
+    winCmapPriv(pColormap);
+
+    /* Call the engine specific colormap destruction procedure */
+    if (!((*pScreenPriv->pwinDestroyColormap) (pColormap))) {
+        winErrorFVerb(2,
+                      "winDestroyColormap - Engine specific colormap destruction "
+                      "procedure failed.  Continuing, but it is possible that memory "
+                      "was leaked, or that colors will be messed up from now on.\n");
     }
 
-  /* Free the colormap privates */
-  free (pCmapPriv);
-  winSetCmapPriv (pColormap, NULL);
+    /* Free the colormap privates */
+    free(pCmapPriv);
+    winSetCmapPriv(pColormap, NULL);
 
 #if CYGDEBUG
-  winDebug ("winDestroyColormap - Returning\n");
+    winDebug("winDestroyColormap - Returning\n");
 #endif
 }
 
-
 /*
  * Internal function to load the palette used by the Shadow DIB
  */
 
 static Bool
-winGetPaletteDIB (ScreenPtr pScreen, ColormapPtr pcmap)
+winGetPaletteDIB(ScreenPtr pScreen, ColormapPtr pcmap)
 {
-  winScreenPriv(pScreen);
-  int			i;
-  Pixel			pixel; /* Pixel == CARD32 */
-  CARD16		nRed, nGreen, nBlue; /* CARD16 == unsigned short */
-  UINT			uiColorsRetrieved = 0;
-  RGBQUAD		rgbColors[WIN_NUM_PALETTE_ENTRIES];
-      
-  /* Get the color table for the screen */
-  uiColorsRetrieved = GetDIBColorTable (pScreenPriv->hdcScreen,
-					0,
-					WIN_NUM_PALETTE_ENTRIES,
-					rgbColors);
-  if (uiColorsRetrieved == 0)
-    {
-      ErrorF ("winGetPaletteDIB - Could not retrieve screen color table\n");
-      return FALSE;
+    winScreenPriv(pScreen);
+    int i;
+    Pixel pixel;                /* Pixel == CARD32 */
+    CARD16 nRed, nGreen, nBlue; /* CARD16 == unsigned short */
+    UINT uiColorsRetrieved = 0;
+    RGBQUAD rgbColors[WIN_NUM_PALETTE_ENTRIES];
+
+    /* Get the color table for the screen */
+    uiColorsRetrieved = GetDIBColorTable(pScreenPriv->hdcScreen,
+                                         0, WIN_NUM_PALETTE_ENTRIES, rgbColors);
+    if (uiColorsRetrieved == 0) {
+        ErrorF("winGetPaletteDIB - Could not retrieve screen color table\n");
+        return FALSE;
     }
 
 #if CYGDEBUG
-  winDebug ("winGetPaletteDIB - Retrieved %d colors from DIB\n",
-	  uiColorsRetrieved);
+    winDebug("winGetPaletteDIB - Retrieved %d colors from DIB\n",
+             uiColorsRetrieved);
 #endif
 
-  /* Set the DIB color table to the default screen palette */
-  if (SetDIBColorTable (pScreenPriv->hdcShadow,
-			0,
-			uiColorsRetrieved,
-			rgbColors) == 0)
-    {
-      ErrorF ("winGetPaletteDIB - SetDIBColorTable () failed\n");
-      return FALSE;
+    /* Set the DIB color table to the default screen palette */
+    if (SetDIBColorTable(pScreenPriv->hdcShadow,
+                         0, uiColorsRetrieved, rgbColors) == 0) {
+        ErrorF("winGetPaletteDIB - SetDIBColorTable () failed\n");
+        return FALSE;
     }
 
-  /* Alloc each color in the DIB color table */
-  for (i = 0; i < uiColorsRetrieved; ++i)
-    {
-      pixel = i;
+    /* Alloc each color in the DIB color table */
+    for (i = 0; i < uiColorsRetrieved; ++i) {
+        pixel = i;
 
-      /* Extract the color values for current palette entry */
-      nRed = rgbColors[i].rgbRed << 8;
-      nGreen = rgbColors[i].rgbGreen << 8;
-      nBlue = rgbColors[i].rgbBlue << 8;
+        /* Extract the color values for current palette entry */
+        nRed = rgbColors[i].rgbRed << 8;
+        nGreen = rgbColors[i].rgbGreen << 8;
+        nBlue = rgbColors[i].rgbBlue << 8;
 
 #if CYGDEBUG
-      winDebug ("winGetPaletteDIB - Allocating a color: %d; "
-	      "%d %d %d\n",
-	      pixel, nRed, nGreen, nBlue);
+        winDebug("winGetPaletteDIB - Allocating a color: %d; "
+                 "%d %d %d\n", pixel, nRed, nGreen, nBlue);
 #endif
 
-      /* Allocate a entry in the X colormap */
-      if (AllocColor (pcmap,
-		      &nRed,
-		      &nGreen,
-		      &nBlue,
-		      &pixel,
-		      0) != Success)
-	{
-	  ErrorF ("winGetPaletteDIB - AllocColor () failed, pixel %d\n",
-		  i);
-	  return FALSE;
-	}
-
-      if (i != pixel
-	  || nRed != rgbColors[i].rgbRed 
-	  || nGreen != rgbColors[i].rgbGreen
-	  || nBlue != rgbColors[i].rgbBlue)
-	{
-	  winDebug ("winGetPaletteDIB - Got: %d; "
-		  "%d %d %d\n",
-		  (int) pixel, nRed, nGreen, nBlue);
-	}
-	  
-      /* FIXME: Not sure that this bit is needed at all */
-      pcmap->red[i].co.local.red = nRed;
-      pcmap->red[i].co.local.green = nGreen;
-      pcmap->red[i].co.local.blue = nBlue;
+        /* Allocate a entry in the X colormap */
+        if (AllocColor(pcmap, &nRed, &nGreen, &nBlue, &pixel, 0) != Success) {
+            ErrorF("winGetPaletteDIB - AllocColor () failed, pixel %d\n", i);
+            return FALSE;
+        }
+
+        if (i != pixel
+            || nRed != rgbColors[i].rgbRed
+            || nGreen != rgbColors[i].rgbGreen
+            || nBlue != rgbColors[i].rgbBlue) {
+            winDebug("winGetPaletteDIB - Got: %d; "
+                     "%d %d %d\n", (int) pixel, nRed, nGreen, nBlue);
+        }
+
+        /* FIXME: Not sure that this bit is needed at all */
+        pcmap->red[i].co.local.red = nRed;
+        pcmap->red[i].co.local.green = nGreen;
+        pcmap->red[i].co.local.blue = nBlue;
     }
 
-  /* System is using a colormap */
-  /* Set the black and white pixel indices */
-  pScreen->whitePixel = uiColorsRetrieved - 1;
-  pScreen->blackPixel = 0;
+    /* System is using a colormap */
+    /* Set the black and white pixel indices */
+    pScreen->whitePixel = uiColorsRetrieved - 1;
+    pScreen->blackPixel = 0;
 
-  return TRUE;
+    return TRUE;
 }
 
-
 /*
  * Internal function to load the standard system palette being used by DD
  */
 
 static Bool
-winGetPaletteDD (ScreenPtr pScreen, ColormapPtr pcmap)
+winGetPaletteDD(ScreenPtr pScreen, ColormapPtr pcmap)
 {
-  int			i;
-  Pixel			pixel; /* Pixel == CARD32 */
-  CARD16		nRed, nGreen, nBlue; /* CARD16 == unsigned short */
-  UINT			uiSystemPaletteEntries;
-  LPPALETTEENTRY	ppeColors = NULL;
-  HDC			hdc = NULL;
-
-  /* Get a DC to obtain the default palette */
-  hdc = GetDC (NULL);
-  if (hdc == NULL)
-    {
-      ErrorF ("winGetPaletteDD - Couldn't get a DC\n");
-      return FALSE;
+    int i;
+    Pixel pixel;                /* Pixel == CARD32 */
+    CARD16 nRed, nGreen, nBlue; /* CARD16 == unsigned short */
+    UINT uiSystemPaletteEntries;
+    LPPALETTEENTRY ppeColors = NULL;
+    HDC hdc = NULL;
+
+    /* Get a DC to obtain the default palette */
+    hdc = GetDC(NULL);
+    if (hdc == NULL) {
+        ErrorF("winGetPaletteDD - Couldn't get a DC\n");
+        return FALSE;
     }
 
-  /* Get the number of entries in the system palette */
-  uiSystemPaletteEntries = GetSystemPaletteEntries (hdc,
-						    0, 0, NULL);
-  if (uiSystemPaletteEntries == 0)
-    {
-      ErrorF ("winGetPaletteDD - Unable to determine number of "
-	      "system palette entries\n");
-      return FALSE;
+    /* Get the number of entries in the system palette */
+    uiSystemPaletteEntries = GetSystemPaletteEntries(hdc, 0, 0, NULL);
+    if (uiSystemPaletteEntries == 0) {
+        ErrorF("winGetPaletteDD - Unable to determine number of "
+               "system palette entries\n");
+        return FALSE;
     }
 
 #if CYGDEBUG
-  winDebug ("winGetPaletteDD - uiSystemPaletteEntries %d\n",
-	  uiSystemPaletteEntries);
+    winDebug("winGetPaletteDD - uiSystemPaletteEntries %d\n",
+             uiSystemPaletteEntries);
 #endif
-  
-  /* Allocate palette entries structure */
-  ppeColors = malloc (uiSystemPaletteEntries * sizeof (PALETTEENTRY));
-  if (ppeColors == NULL)
-    {
-      ErrorF ("winGetPaletteDD - malloc () for colormap failed\n");
-      return FALSE;
+
+    /* Allocate palette entries structure */
+    ppeColors = malloc(uiSystemPaletteEntries * sizeof(PALETTEENTRY));
+    if (ppeColors == NULL) {
+        ErrorF("winGetPaletteDD - malloc () for colormap failed\n");
+        return FALSE;
     }
 
-  /* Get system palette entries */
-  GetSystemPaletteEntries (hdc,
-			   0, uiSystemPaletteEntries, ppeColors);
+    /* Get system palette entries */
+    GetSystemPaletteEntries(hdc, 0, uiSystemPaletteEntries, ppeColors);
 
-  /* Allocate an X colormap entry for every system palette entry */
-  for (i = 0; i < uiSystemPaletteEntries; ++i)
-    {
-      pixel = i;
+    /* Allocate an X colormap entry for every system palette entry */
+    for (i = 0; i < uiSystemPaletteEntries; ++i) {
+        pixel = i;
 
-      /* Extract the color values for current palette entry */
-      nRed = ppeColors[i].peRed << 8;
-      nGreen = ppeColors[i].peGreen << 8;
-      nBlue = ppeColors[i].peBlue << 8;
+        /* Extract the color values for current palette entry */
+        nRed = ppeColors[i].peRed << 8;
+        nGreen = ppeColors[i].peGreen << 8;
+        nBlue = ppeColors[i].peBlue << 8;
 #if CYGDEBUG
-      winDebug ("winGetPaletteDD - Allocating a color: %d; "
-	      "%d %d %d\n",
-	      pixel, nRed, nGreen, nBlue);
+        winDebug("winGetPaletteDD - Allocating a color: %d; "
+                 "%d %d %d\n", pixel, nRed, nGreen, nBlue);
 #endif
-      if (AllocColor (pcmap,
-		      &nRed,
-		      &nGreen,
-		      &nBlue,
-		      &pixel,
-		      0) != Success)
-	{
-	  ErrorF ("winGetPaletteDD - AllocColor () failed, pixel %d\n",
-		  i);
-	  free (ppeColors);
-	  ppeColors = NULL;
-	  return FALSE;
-	}
-
-      pcmap->red[i].co.local.red = nRed;
-      pcmap->red[i].co.local.green = nGreen;
-      pcmap->red[i].co.local.blue = nBlue;
+        if (AllocColor(pcmap, &nRed, &nGreen, &nBlue, &pixel, 0) != Success) {
+            ErrorF("winGetPaletteDD - AllocColor () failed, pixel %d\n", i);
+            free(ppeColors);
+            ppeColors = NULL;
+            return FALSE;
+        }
+
+        pcmap->red[i].co.local.red = nRed;
+        pcmap->red[i].co.local.green = nGreen;
+        pcmap->red[i].co.local.blue = nBlue;
     }
 
-  /* System is using a colormap */
-  /* Set the black and white pixel indices */
-  pScreen->whitePixel = uiSystemPaletteEntries - 1;
-  pScreen->blackPixel = 0;
+    /* System is using a colormap */
+    /* Set the black and white pixel indices */
+    pScreen->whitePixel = uiSystemPaletteEntries - 1;
+    pScreen->blackPixel = 0;
 
-  /* Free colormap */
-  free(ppeColors);
-  ppeColors = NULL;
+    /* Free colormap */
+    free(ppeColors);
+    ppeColors = NULL;
 
-  /* Free the DC */
-  if (hdc != NULL)
-    {
-      ReleaseDC (NULL, hdc);
-      hdc = NULL;
+    /* Free the DC */
+    if (hdc != NULL) {
+        ReleaseDC(NULL, hdc);
+        hdc = NULL;
     }
 
-  return TRUE;
+    return TRUE;
 }
 
-
 /*
  * Install the standard fb colormap, or the GDI colormap,
  * depending on the current screen depth.
  */
 
 Bool
-winCreateDefColormap (ScreenPtr pScreen)
+winCreateDefColormap(ScreenPtr pScreen)
 {
-  winScreenPriv(pScreen);
-  winScreenInfo		*pScreenInfo = pScreenPriv->pScreenInfo;
-  unsigned short	zero = 0, ones = 0xFFFF;
-  VisualPtr		pVisual = pScreenPriv->pRootVisual;
-  ColormapPtr		pcmap = NULL;
-  Pixel			wp, bp;
+    winScreenPriv(pScreen);
+    winScreenInfo *pScreenInfo = pScreenPriv->pScreenInfo;
+    unsigned short zero = 0, ones = 0xFFFF;
+    VisualPtr pVisual = pScreenPriv->pRootVisual;
+    ColormapPtr pcmap = NULL;
+    Pixel wp, bp;
 
 #if CYGDEBUG
-  winDebug ("winCreateDefColormap\n");
+    winDebug("winCreateDefColormap\n");
 #endif
 
-  /* Use standard fb colormaps for non palettized color modes */
-  if (pScreenInfo->dwBPP > 8)
-    {
-      winDebug ("winCreateDefColormap - Deferring to " \
-	      "fbCreateDefColormap ()\n");
-      return fbCreateDefColormap (pScreen);
+    /* Use standard fb colormaps for non palettized color modes */
+    if (pScreenInfo->dwBPP > 8) {
+        winDebug("winCreateDefColormap - Deferring to "
+                 "fbCreateDefColormap ()\n");
+        return fbCreateDefColormap(pScreen);
     }
 
-  /*
-   *  AllocAll for non-Dynamic visual classes,
-   *  AllocNone for Dynamic visual classes.
-   */
+    /*
+     *  AllocAll for non-Dynamic visual classes,
+     *  AllocNone for Dynamic visual classes.
+     */
 
-  /*
-   * Dynamic visual classes allow the colors of the color map
-   * to be changed by clients.
-   */
+    /*
+     * Dynamic visual classes allow the colors of the color map
+     * to be changed by clients.
+     */
 
 #if CYGDEBUG
-  winDebug ("winCreateDefColormap - defColormap: %d\n",
-	  pScreen->defColormap);
+    winDebug("winCreateDefColormap - defColormap: %d\n", pScreen->defColormap);
 #endif
 
-  /* Allocate an X colormap, owned by client 0 */
-  if (CreateColormap (pScreen->defColormap, 
-		      pScreen,
-		      pVisual,
-		      &pcmap,
-		      (pVisual->class & DynamicClass) ? AllocNone : AllocAll,
-		      0) != Success)
-    {
-      ErrorF ("winCreateDefColormap - CreateColormap failed\n");
-      return FALSE;
+    /* Allocate an X colormap, owned by client 0 */
+    if (CreateColormap(pScreen->defColormap,
+                       pScreen,
+                       pVisual,
+                       &pcmap,
+                       (pVisual->class & DynamicClass) ? AllocNone : AllocAll,
+                       0) != Success) {
+        ErrorF("winCreateDefColormap - CreateColormap failed\n");
+        return FALSE;
     }
-  if (pcmap == NULL)
-    {
-      ErrorF ("winCreateDefColormap - Colormap could not be created\n");
-      return FALSE;
+    if (pcmap == NULL) {
+        ErrorF("winCreateDefColormap - Colormap could not be created\n");
+        return FALSE;
     }
 
 #if CYGDEBUG
-  winDebug ("winCreateDefColormap - Created a colormap\n");
+    winDebug("winCreateDefColormap - Created a colormap\n");
 #endif
 
-  /* Branch on the visual class */
-  if (!(pVisual->class & DynamicClass))
-    {
-      /* Branch on engine type */
-      if (pScreenInfo->dwEngine == WIN_SERVER_SHADOW_GDI)
-	{
-	  /* Load the colors being used by the Shadow DIB */
-	  if (!winGetPaletteDIB (pScreen, pcmap))
-	    {
-	      ErrorF ("winCreateDefColormap - Couldn't get DIB colors\n");
-	      return FALSE;
-	    }
-	}
-      else
-	{
-	  /* Load the colors from the default system palette */
-	  if (!winGetPaletteDD (pScreen, pcmap))
-	    {
-	      ErrorF ("winCreateDefColormap - Couldn't get colors "
-		      "for DD\n");
-	      return FALSE;
-	    }
-	}
+    /* Branch on the visual class */
+    if (!(pVisual->class & DynamicClass)) {
+        /* Branch on engine type */
+        if (pScreenInfo->dwEngine == WIN_SERVER_SHADOW_GDI) {
+            /* Load the colors being used by the Shadow DIB */
+            if (!winGetPaletteDIB(pScreen, pcmap)) {
+                ErrorF("winCreateDefColormap - Couldn't get DIB colors\n");
+                return FALSE;
+            }
+        }
+        else {
+            /* Load the colors from the default system palette */
+            if (!winGetPaletteDD(pScreen, pcmap)) {
+                ErrorF("winCreateDefColormap - Couldn't get colors "
+                       "for DD\n");
+                return FALSE;
+            }
+        }
     }
-  else
-    {
-      wp = pScreen->whitePixel;
-      bp = pScreen->blackPixel;
-      
-      /* Allocate a black and white pixel */
-      if ((AllocColor (pcmap, &ones, &ones, &ones, &wp, 0) !=
-	   Success)
-	  ||
-	  (AllocColor (pcmap, &zero, &zero, &zero, &bp, 0) !=
-	   Success))
-	{
-	  ErrorF ("winCreateDefColormap - Couldn't allocate bp or wp\n");
-	  return FALSE;
-	}
-      
-      pScreen->whitePixel = wp;
-      pScreen->blackPixel = bp;
+    else {
+        wp = pScreen->whitePixel;
+        bp = pScreen->blackPixel;
+
+        /* Allocate a black and white pixel */
+        if ((AllocColor(pcmap, &ones, &ones, &ones, &wp, 0) != Success)
+            || (AllocColor(pcmap, &zero, &zero, &zero, &bp, 0) != Success)) {
+            ErrorF("winCreateDefColormap - Couldn't allocate bp or wp\n");
+            return FALSE;
+        }
+
+        pScreen->whitePixel = wp;
+        pScreen->blackPixel = bp;
 
 #if 0
-      /* Have to reserve first 10 and last ten pixels in DirectDraw windowed */
-      if (pScreenInfo->dwEngine != WIN_SERVER_SHADOW_GDI)
-	{
-	  int		k;
-	  Pixel		p;
-
-	  for (k = 1; k < 10; ++k)
-	    {
-	      p = k;
-	      if (AllocColor (pcmap, &ones, &ones, &ones, &p, 0) != Success)
-		FatalError ("Foo!\n");
-	    }
-	  
-	  for (k = 245; k < 255; ++k)
-	    {
-	      p = k;
-	      if (AllocColor (pcmap, &zero, &zero, &zero, &p, 0) != Success)
-		FatalError ("Baz!\n");
-	    }
-	}
+        /* Have to reserve first 10 and last ten pixels in DirectDraw windowed */
+        if (pScreenInfo->dwEngine != WIN_SERVER_SHADOW_GDI) {
+            int k;
+            Pixel p;
+
+            for (k = 1; k < 10; ++k) {
+                p = k;
+                if (AllocColor(pcmap, &ones, &ones, &ones, &p, 0) != Success)
+                    FatalError("Foo!\n");
+            }
+
+            for (k = 245; k < 255; ++k) {
+                p = k;
+                if (AllocColor(pcmap, &zero, &zero, &zero, &p, 0) != Success)
+                    FatalError("Baz!\n");
+            }
+        }
 #endif
     }
 
-  /* Install the created colormap */
-  (*pScreen->InstallColormap)(pcmap);
+    /* Install the created colormap */
+    (*pScreen->InstallColormap) (pcmap);
 
 #if CYGDEBUG
-  winDebug ("winCreateDefColormap - Returning\n");
+    winDebug("winCreateDefColormap - Returning\n");
 #endif
 
-  return TRUE;
+    return TRUE;
 }