diff src/video/dga/SDL_dgavideo.c @ 1668:4da1ee79c9af SDL-1.3

more tweaking indent options
author Sam Lantinga <slouken@libsdl.org>
date Mon, 29 May 2006 04:04:35 +0000
parents 782fd950bd46
children
line wrap: on
line diff
--- a/src/video/dga/SDL_dgavideo.c	Mon May 29 03:53:21 2006 +0000
+++ b/src/video/dga/SDL_dgavideo.c	Mon May 29 04:04:35 2006 +0000
@@ -44,40 +44,40 @@
 /*#define DGA_DEBUG*/
 
 /* Heheh we're using X11 event code */
-extern void X11_SaveScreenSaver (Display * display, int *saved_timeout,
-                                 BOOL * dpms);
-extern void X11_DisableScreenSaver (Display * display);
-extern void X11_RestoreScreenSaver (Display * display, int saved_timeout,
-                                    BOOL dpms);
+extern void X11_SaveScreenSaver(Display * display, int *saved_timeout,
+                                BOOL * dpms);
+extern void X11_DisableScreenSaver(Display * display);
+extern void X11_RestoreScreenSaver(Display * display, int saved_timeout,
+                                   BOOL dpms);
 
 /* Initialization/Query functions */
-static int DGA_VideoInit (_THIS, SDL_PixelFormat * vformat);
-static SDL_Rect **DGA_ListModes (_THIS, SDL_PixelFormat * format,
-                                 Uint32 flags);
-static SDL_Surface *DGA_SetVideoMode (_THIS, SDL_Surface * current, int width,
-                                      int height, int bpp, Uint32 flags);
-static int DGA_SetColors (_THIS, int firstcolor, int ncolors,
-                          SDL_Color * colors);
-static int DGA_SetGammaRamp (_THIS, Uint16 * ramp);
-static void DGA_VideoQuit (_THIS);
+static int DGA_VideoInit(_THIS, SDL_PixelFormat * vformat);
+static SDL_Rect **DGA_ListModes(_THIS, SDL_PixelFormat * format,
+                                Uint32 flags);
+static SDL_Surface *DGA_SetVideoMode(_THIS, SDL_Surface * current, int width,
+                                     int height, int bpp, Uint32 flags);
+static int DGA_SetColors(_THIS, int firstcolor, int ncolors,
+                         SDL_Color * colors);
+static int DGA_SetGammaRamp(_THIS, Uint16 * ramp);
+static void DGA_VideoQuit(_THIS);
 
 /* Hardware surface functions */
-static int DGA_InitHWSurfaces (_THIS, SDL_Surface * screen, Uint8 * base,
-                               int size);
-static void DGA_FreeHWSurfaces (_THIS);
-static int DGA_AllocHWSurface (_THIS, SDL_Surface * surface);
-static int DGA_FillHWRect (_THIS, SDL_Surface * dst, SDL_Rect * rect,
-                           Uint32 color);
-static int DGA_CheckHWBlit (_THIS, SDL_Surface * src, SDL_Surface * dst);
-static int DGA_LockHWSurface (_THIS, SDL_Surface * surface);
-static void DGA_UnlockHWSurface (_THIS, SDL_Surface * surface);
-static void DGA_FreeHWSurface (_THIS, SDL_Surface * surface);
-static int DGA_FlipHWSurface (_THIS, SDL_Surface * surface);
+static int DGA_InitHWSurfaces(_THIS, SDL_Surface * screen, Uint8 * base,
+                              int size);
+static void DGA_FreeHWSurfaces(_THIS);
+static int DGA_AllocHWSurface(_THIS, SDL_Surface * surface);
+static int DGA_FillHWRect(_THIS, SDL_Surface * dst, SDL_Rect * rect,
+                          Uint32 color);
+static int DGA_CheckHWBlit(_THIS, SDL_Surface * src, SDL_Surface * dst);
+static int DGA_LockHWSurface(_THIS, SDL_Surface * surface);
+static void DGA_UnlockHWSurface(_THIS, SDL_Surface * surface);
+static void DGA_FreeHWSurface(_THIS, SDL_Surface * surface);
+static int DGA_FlipHWSurface(_THIS, SDL_Surface * surface);
 
 /* DGA driver bootstrap functions */
 
 static int
-DGA_Available (void)
+DGA_Available(void)
 {
     const char *display = NULL;
     Display *dpy = NULL;
@@ -86,65 +86,65 @@
     /* The driver is available is available if the display is local
        and the DGA 2.0+ extension is available, and we can map mem.
      */
-    if (SDL_X11_LoadSymbols ()) {
-        if ((SDL_strncmp (XDisplayName (display), ":", 1) == 0) ||
-            (SDL_strncmp (XDisplayName (display), "unix:", 5) == 0)) {
-            dpy = XOpenDisplay (display);
+    if (SDL_X11_LoadSymbols()) {
+        if ((SDL_strncmp(XDisplayName(display), ":", 1) == 0) ||
+            (SDL_strncmp(XDisplayName(display), "unix:", 5) == 0)) {
+            dpy = XOpenDisplay(display);
             if (dpy) {
                 int events, errors, major, minor;
 
-                if (SDL_NAME (XDGAQueryExtension)
+                if (SDL_NAME(XDGAQueryExtension)
                     (dpy, &events, &errors)
-                    && SDL_NAME (XDGAQueryVersion) (dpy, &major, &minor)) {
+                    && SDL_NAME(XDGAQueryVersion) (dpy, &major, &minor)) {
                     int screen;
 
-                    screen = DefaultScreen (dpy);
+                    screen = DefaultScreen(dpy);
                     if ((major >= 2) &&
-                        SDL_NAME (XDGAOpenFramebuffer) (dpy, screen)) {
+                        SDL_NAME(XDGAOpenFramebuffer) (dpy, screen)) {
                         available = 1;
-                        SDL_NAME (XDGACloseFramebuffer) (dpy, screen);
+                        SDL_NAME(XDGACloseFramebuffer) (dpy, screen);
                     }
                 }
-                XCloseDisplay (dpy);
+                XCloseDisplay(dpy);
             }
         }
-        SDL_X11_UnloadSymbols ();
+        SDL_X11_UnloadSymbols();
     }
     return (available);
 }
 
 static void
-DGA_DeleteDevice (SDL_VideoDevice * device)
+DGA_DeleteDevice(SDL_VideoDevice * device)
 {
     if (device != NULL) {
-        SDL_free (device->hidden);
-        SDL_free (device);
-        SDL_X11_UnloadSymbols ();
+        SDL_free(device->hidden);
+        SDL_free(device);
+        SDL_X11_UnloadSymbols();
     }
 }
 
 static SDL_VideoDevice *
-DGA_CreateDevice (int devindex)
+DGA_CreateDevice(int devindex)
 {
     SDL_VideoDevice *device = NULL;
 
     /* Initialize all variables that we clean on shutdown */
-    if (SDL_X11_LoadSymbols ()) {
-        device = (SDL_VideoDevice *) SDL_malloc (sizeof (SDL_VideoDevice));
+    if (SDL_X11_LoadSymbols()) {
+        device = (SDL_VideoDevice *) SDL_malloc(sizeof(SDL_VideoDevice));
         if (device) {
-            SDL_memset (device, 0, (sizeof *device));
+            SDL_memset(device, 0, (sizeof *device));
             device->hidden = (struct SDL_PrivateVideoData *)
-                SDL_malloc ((sizeof *device->hidden));
+                SDL_malloc((sizeof *device->hidden));
         }
         if ((device == NULL) || (device->hidden == NULL)) {
-            SDL_OutOfMemory ();
+            SDL_OutOfMemory();
             if (device) {
-                SDL_free (device);
+                SDL_free(device);
             }
-            SDL_X11_UnloadSymbols ();
+            SDL_X11_UnloadSymbols();
             return (0);
         }
-        SDL_memset (device->hidden, 0, (sizeof *device->hidden));
+        SDL_memset(device->hidden, 0, (sizeof *device->hidden));
 
         /* Set the function pointers */
         device->VideoInit = DGA_VideoInit;
@@ -184,7 +184,7 @@
 };
 
 static int
-DGA_AddMode (_THIS, int bpp, int w, int h)
+DGA_AddMode(_THIS, int bpp, int w, int h)
 {
     SDL_Rect *mode;
     int index;
@@ -203,9 +203,9 @@
     }
 
     /* Set up the new video mode rectangle */
-    mode = (SDL_Rect *) SDL_malloc (sizeof *mode);
+    mode = (SDL_Rect *) SDL_malloc(sizeof *mode);
     if (mode == NULL) {
-        SDL_OutOfMemory ();
+        SDL_OutOfMemory();
         return (-1);
     }
     mode->x = 0;
@@ -216,12 +216,12 @@
     /* Allocate the new list of modes, and fill in the new mode */
     next_mode = SDL_nummodes[index];
     SDL_modelist[index] = (SDL_Rect **)
-        SDL_realloc (SDL_modelist[index],
-                     (1 + next_mode + 1) * sizeof (SDL_Rect *));
+        SDL_realloc(SDL_modelist[index],
+                    (1 + next_mode + 1) * sizeof(SDL_Rect *));
     if (SDL_modelist[index] == NULL) {
-        SDL_OutOfMemory ();
+        SDL_OutOfMemory();
         SDL_nummodes[index] = 0;
-        SDL_free (mode);
+        SDL_free(mode);
         return (-1);
     }
     SDL_modelist[index][next_mode] = mode;
@@ -233,75 +233,75 @@
 
 /* This whole function is a hack. :) */
 static Uint32
-get_video_size (_THIS)
+get_video_size(_THIS)
 {
     /* This is a non-exported function from libXxf86dga.a */
-    extern unsigned char *SDL_NAME (XDGAGetMappedMemory) (int screen);
+    extern unsigned char *SDL_NAME(XDGAGetMappedMemory) (int screen);
     FILE *proc;
     unsigned long mem;
     unsigned start, stop;
     char line[BUFSIZ];
     Uint32 size;
 
-    mem = (unsigned long) SDL_NAME (XDGAGetMappedMemory) (DGA_Screen);
+    mem = (unsigned long) SDL_NAME(XDGAGetMappedMemory) (DGA_Screen);
     size = 0;
-    proc = fopen ("/proc/self/maps", "r");
+    proc = fopen("/proc/self/maps", "r");
     if (proc) {
-        while (fgets (line, sizeof (line) - 1, proc)) {
-            SDL_sscanf (line, "%x-%x", &start, &stop);
+        while (fgets(line, sizeof(line) - 1, proc)) {
+            SDL_sscanf(line, "%x-%x", &start, &stop);
             if (start == mem) {
                 size = (Uint32) ((stop - start) / 1024);
                 break;
             }
         }
-        fclose (proc);
+        fclose(proc);
     }
     return (size);
 }
 
 #ifdef DGA_DEBUG
 static void
-PrintMode (SDL_NAME (XDGAMode) * mode)
+PrintMode(SDL_NAME(XDGAMode) * mode)
 {
-    printf ("Mode: %s (%dx%d) at %d bpp (%f refresh, %d pitch) num: %d\n",
-            mode->name,
-            mode->viewportWidth, mode->viewportHeight,
-            mode->depth == 24 ? mode->bitsPerPixel : mode->depth,
-            mode->verticalRefresh, mode->bytesPerScanline, mode->num);
-    printf ("\tRGB: 0x%8.8x 0x%8.8x 0x%8.8x (%d - %s)\n",
-            mode->redMask, mode->greenMask, mode->blueMask,
-            mode->visualClass,
-            mode->visualClass == TrueColor ? "truecolor" :
-            mode->visualClass == DirectColor ? "directcolor" :
-            mode->visualClass == PseudoColor ? "pseudocolor" : "unknown");
-    printf ("\tFlags: ");
+    printf("Mode: %s (%dx%d) at %d bpp (%f refresh, %d pitch) num: %d\n",
+           mode->name,
+           mode->viewportWidth, mode->viewportHeight,
+           mode->depth == 24 ? mode->bitsPerPixel : mode->depth,
+           mode->verticalRefresh, mode->bytesPerScanline, mode->num);
+    printf("\tRGB: 0x%8.8x 0x%8.8x 0x%8.8x (%d - %s)\n",
+           mode->redMask, mode->greenMask, mode->blueMask,
+           mode->visualClass,
+           mode->visualClass == TrueColor ? "truecolor" :
+           mode->visualClass == DirectColor ? "directcolor" :
+           mode->visualClass == PseudoColor ? "pseudocolor" : "unknown");
+    printf("\tFlags: ");
     if (mode->flags & XDGAConcurrentAccess)
-        printf (" XDGAConcurrentAccess");
+        printf(" XDGAConcurrentAccess");
     if (mode->flags & XDGASolidFillRect)
-        printf (" XDGASolidFillRect");
+        printf(" XDGASolidFillRect");
     if (mode->flags & XDGABlitRect)
-        printf (" XDGABlitRect");
+        printf(" XDGABlitRect");
     if (mode->flags & XDGABlitTransRect)
-        printf (" XDGABlitTransRect");
+        printf(" XDGABlitTransRect");
     if (mode->flags & XDGAPixmap)
-        printf (" XDGAPixmap");
+        printf(" XDGAPixmap");
     if (mode->flags & XDGAInterlaced)
-        printf (" XDGAInterlaced");
+        printf(" XDGAInterlaced");
     if (mode->flags & XDGADoublescan)
-        printf (" XDGADoublescan");
+        printf(" XDGADoublescan");
     if (mode->viewportFlags & XDGAFlipRetrace)
-        printf (" XDGAFlipRetrace");
+        printf(" XDGAFlipRetrace");
     if (mode->viewportFlags & XDGAFlipImmediate)
-        printf (" XDGAFlipImmediate");
-    printf ("\n");
+        printf(" XDGAFlipImmediate");
+    printf("\n");
 }
 #endif /* DGA_DEBUG */
 
 static int
-cmpmodes (const void *va, const void *vb)
+cmpmodes(const void *va, const void *vb)
 {
-    const SDL_NAME (XDGAMode) * a = (const SDL_NAME (XDGAMode) *) va;
-    const SDL_NAME (XDGAMode) * b = (const SDL_NAME (XDGAMode) *) vb;
+    const SDL_NAME(XDGAMode) * a = (const SDL_NAME(XDGAMode) *) va;
+    const SDL_NAME(XDGAMode) * b = (const SDL_NAME(XDGAMode) *) vb;
 
     if ((a->viewportWidth == b->viewportWidth) &&
         (b->viewportHeight == a->viewportHeight)) {
@@ -325,7 +325,7 @@
     }
 }
 static void
-UpdateHWInfo (_THIS, SDL_NAME (XDGAMode) * mode)
+UpdateHWInfo(_THIS, SDL_NAME(XDGAMode) * mode)
 {
     this->info.wm_available = 0;
     this->info.hw_available = 1;
@@ -344,58 +344,58 @@
     } else {
         this->info.blit_fill = 0;
     }
-    this->info.video_mem = get_video_size (this);
+    this->info.video_mem = get_video_size(this);
 }
 
 static int
-DGA_VideoInit (_THIS, SDL_PixelFormat * vformat)
+DGA_VideoInit(_THIS, SDL_PixelFormat * vformat)
 {
     const char *display;
     int event_base, error_base;
     int major_version, minor_version;
     Visual *visual;
-    SDL_NAME (XDGAMode) * modes;
+    SDL_NAME(XDGAMode) * modes;
     int i, num_modes;
 
     /* Open the X11 display */
     display = NULL;             /* Get it from DISPLAY environment variable */
 
-    DGA_Display = XOpenDisplay (display);
+    DGA_Display = XOpenDisplay(display);
     if (DGA_Display == NULL) {
-        SDL_SetError ("Couldn't open X11 display");
+        SDL_SetError("Couldn't open X11 display");
         return (-1);
     }
 
     /* Check for the DGA extension */
-    if (!SDL_NAME (XDGAQueryExtension) (DGA_Display, &event_base, &error_base)
-        || !SDL_NAME (XDGAQueryVersion) (DGA_Display, &major_version,
-                                         &minor_version)) {
-        SDL_SetError ("DGA extension not available");
-        XCloseDisplay (DGA_Display);
+    if (!SDL_NAME(XDGAQueryExtension) (DGA_Display, &event_base, &error_base)
+        || !SDL_NAME(XDGAQueryVersion) (DGA_Display, &major_version,
+                                        &minor_version)) {
+        SDL_SetError("DGA extension not available");
+        XCloseDisplay(DGA_Display);
         return (-1);
     }
     if (major_version < 2) {
-        SDL_SetError ("DGA driver requires DGA 2.0 or newer");
-        XCloseDisplay (DGA_Display);
+        SDL_SetError("DGA driver requires DGA 2.0 or newer");
+        XCloseDisplay(DGA_Display);
         return (-1);
     }
     DGA_event_base = event_base;
 
     /* Determine the current screen size */
-    this->info.current_w = DisplayWidth (DGA_Display, DGA_Screen);
-    this->info.current_h = DisplayHeight (DGA_Display, DGA_Screen);
+    this->info.current_w = DisplayWidth(DGA_Display, DGA_Screen);
+    this->info.current_h = DisplayHeight(DGA_Display, DGA_Screen);
 
     /* Determine the current screen depth */
-    visual = DefaultVisual (DGA_Display, DGA_Screen);
+    visual = DefaultVisual(DGA_Display, DGA_Screen);
     {
         XPixmapFormatValues *pix_format;
         int i, num_formats;
 
-        vformat->BitsPerPixel = DefaultDepth (DGA_Display, DGA_Screen);
-        pix_format = XListPixmapFormats (DGA_Display, &num_formats);
+        vformat->BitsPerPixel = DefaultDepth(DGA_Display, DGA_Screen);
+        pix_format = XListPixmapFormats(DGA_Display, &num_formats);
         if (pix_format == NULL) {
-            SDL_SetError ("Couldn't determine screen formats");
-            XCloseDisplay (DGA_Display);
+            SDL_SetError("Couldn't determine screen formats");
+            XCloseDisplay(DGA_Display);
             return (-1);
         }
         for (i = 0; i < num_formats; ++i) {
@@ -404,7 +404,7 @@
         }
         if (i != num_formats)
             vformat->BitsPerPixel = pix_format[i].bits_per_pixel;
-        XFree ((char *) pix_format);
+        XFree((char *) pix_format);
     }
     if (vformat->BitsPerPixel > 8) {
         vformat->Rmask = visual->red_mask;
@@ -413,44 +413,44 @@
     }
 
     /* Open access to the framebuffer */
-    if (!SDL_NAME (XDGAOpenFramebuffer) (DGA_Display, DGA_Screen)) {
-        SDL_SetError ("Unable to map the video memory");
-        XCloseDisplay (DGA_Display);
+    if (!SDL_NAME(XDGAOpenFramebuffer) (DGA_Display, DGA_Screen)) {
+        SDL_SetError("Unable to map the video memory");
+        XCloseDisplay(DGA_Display);
         return (-1);
     }
 
     /* Save DPMS and screensaver settings */
-    X11_SaveScreenSaver (DGA_Display, &screensaver_timeout, &dpms_enabled);
-    X11_DisableScreenSaver (DGA_Display);
+    X11_SaveScreenSaver(DGA_Display, &screensaver_timeout, &dpms_enabled);
+    X11_DisableScreenSaver(DGA_Display);
 
     /* Query for the list of available video modes */
-    modes = SDL_NAME (XDGAQueryModes) (DGA_Display, DGA_Screen, &num_modes);
-    SDL_qsort (modes, num_modes, sizeof *modes, cmpmodes);
+    modes = SDL_NAME(XDGAQueryModes) (DGA_Display, DGA_Screen, &num_modes);
+    SDL_qsort(modes, num_modes, sizeof *modes, cmpmodes);
     for (i = 0; i < num_modes; ++i) {
         if (((modes[i].visualClass == PseudoColor) ||
              (modes[i].visualClass == DirectColor) ||
              (modes[i].visualClass == TrueColor)) &&
             !(modes[i].flags & (XDGAInterlaced | XDGADoublescan))) {
 #ifdef DGA_DEBUG
-            PrintMode (&modes[i]);
+            PrintMode(&modes[i]);
 #endif
-            DGA_AddMode (this, modes[i].bitsPerPixel,
-                         modes[i].viewportWidth, modes[i].viewportHeight);
+            DGA_AddMode(this, modes[i].bitsPerPixel,
+                        modes[i].viewportWidth, modes[i].viewportHeight);
         }
     }
-    UpdateHWInfo (this, modes);
-    XFree (modes);
+    UpdateHWInfo(this, modes);
+    XFree(modes);
 
     /* Create the hardware surface lock mutex */
-    hw_lock = SDL_CreateMutex ();
+    hw_lock = SDL_CreateMutex();
     if (hw_lock == NULL) {
-        SDL_SetError ("Unable to create lock mutex");
-        DGA_VideoQuit (this);
+        SDL_SetError("Unable to create lock mutex");
+        DGA_VideoQuit(this);
         return (-1);
     }
 #ifdef LOCK_DGA_DISPLAY
     /* Create the event lock so we're thread-safe.. :-/ */
-    event_lock = SDL_CreateMutex ();
+    event_lock = SDL_CreateMutex();
 #endif /* LOCK_DGA_DISPLAY */
 
     /* We're done! */
@@ -458,34 +458,34 @@
 }
 
 SDL_Rect **
-DGA_ListModes (_THIS, SDL_PixelFormat * format, Uint32 flags)
+DGA_ListModes(_THIS, SDL_PixelFormat * format, Uint32 flags)
 {
     return (SDL_modelist[((format->BitsPerPixel + 7) / 8) - 1]);
 }
 
 /* Various screen update functions available */
-static void DGA_DirectUpdate (_THIS, int numrects, SDL_Rect * rects);
+static void DGA_DirectUpdate(_THIS, int numrects, SDL_Rect * rects);
 
 SDL_Surface *
-DGA_SetVideoMode (_THIS, SDL_Surface * current,
-                  int width, int height, int bpp, Uint32 flags)
+DGA_SetVideoMode(_THIS, SDL_Surface * current,
+                 int width, int height, int bpp, Uint32 flags)
 {
-    SDL_NAME (XDGAMode) * modes;
+    SDL_NAME(XDGAMode) * modes;
     int i, num_modes;
-    SDL_NAME (XDGADevice) * mode;
+    SDL_NAME(XDGADevice) * mode;
     int screen_len;
     Uint8 *surfaces_mem;
     int surfaces_len;
 
     /* Free any previous colormap */
     if (DGA_colormap) {
-        XFreeColormap (DGA_Display, DGA_colormap);
+        XFreeColormap(DGA_Display, DGA_colormap);
         DGA_colormap = 0;
     }
 
     /* Search for a matching video mode */
-    modes = SDL_NAME (XDGAQueryModes) (DGA_Display, DGA_Screen, &num_modes);
-    SDL_qsort (modes, num_modes, sizeof *modes, cmpmodes);
+    modes = SDL_NAME(XDGAQueryModes) (DGA_Display, DGA_Screen, &num_modes);
+    SDL_qsort(modes, num_modes, sizeof *modes, cmpmodes);
     for (i = 0; i < num_modes; ++i) {
         int depth;
 
@@ -504,18 +504,18 @@
         }
     }
     if (i == num_modes) {
-        SDL_SetError ("No matching video mode found");
+        SDL_SetError("No matching video mode found");
         return (NULL);
     }
 #ifdef DGA_DEBUG
-    PrintMode (&modes[i]);
+    PrintMode(&modes[i]);
 #endif
 
     /* Set the video mode */
-    mode = SDL_NAME (XDGASetMode) (DGA_Display, DGA_Screen, modes[i].num);
-    XFree (modes);
+    mode = SDL_NAME(XDGASetMode) (DGA_Display, DGA_Screen, modes[i].num);
+    XFree(modes);
     if (mode == NULL) {
-        SDL_SetError ("Unable to switch to requested mode");
+        SDL_SetError("Unable to switch to requested mode");
         return (NULL);
     }
     DGA_visualClass = mode->mode.visualClass;
@@ -528,9 +528,9 @@
     current->h = mode->mode.viewportHeight;
     current->pitch = memory_pitch;
     current->pixels = memory_base;
-    if (!SDL_ReallocFormat (current, mode->mode.bitsPerPixel,
-                            mode->mode.redMask,
-                            mode->mode.greenMask, mode->mode.blueMask, 0)) {
+    if (!SDL_ReallocFormat(current, mode->mode.bitsPerPixel,
+                           mode->mode.redMask,
+                           mode->mode.greenMask, mode->mode.blueMask, 0)) {
         return (NULL);
     }
     screen_len = current->h * current->pitch;
@@ -538,34 +538,34 @@
     /* Create a colormap if necessary */
     if ((DGA_visualClass == PseudoColor) || (DGA_visualClass == DirectColor)) {
         DGA_colormap =
-            SDL_NAME (XDGACreateColormap) (DGA_Display, DGA_Screen, mode,
-                                           AllocAll);
+            SDL_NAME(XDGACreateColormap) (DGA_Display, DGA_Screen, mode,
+                                          AllocAll);
         if (DGA_visualClass == PseudoColor) {
             current->flags |= SDL_HWPALETTE;
         } else {
             /* Initialize the colormap to the identity mapping */
-            SDL_GetGammaRamp (0, 0, 0);
+            SDL_GetGammaRamp(0, 0, 0);
             this->screen = current;
-            DGA_SetGammaRamp (this, this->gamma);
+            DGA_SetGammaRamp(this, this->gamma);
             this->screen = NULL;
         }
     } else {
         DGA_colormap =
-            SDL_NAME (XDGACreateColormap) (DGA_Display, DGA_Screen, mode,
-                                           AllocNone);
+            SDL_NAME(XDGACreateColormap) (DGA_Display, DGA_Screen, mode,
+                                          AllocNone);
     }
-    SDL_NAME (XDGAInstallColormap) (DGA_Display, DGA_Screen, DGA_colormap);
+    SDL_NAME(XDGAInstallColormap) (DGA_Display, DGA_Screen, DGA_colormap);
 
     /* Update the hardware capabilities */
-    UpdateHWInfo (this, &mode->mode);
+    UpdateHWInfo(this, &mode->mode);
 
     /* Set up the information for hardware surfaces */
     surfaces_mem = (Uint8 *) current->pixels + screen_len;
     surfaces_len = (mode->mode.imageHeight * current->pitch - screen_len);
 
     /* Update for double-buffering, if we can */
-    SDL_NAME (XDGASetViewport) (DGA_Display, DGA_Screen, 0, 0,
-                                XDGAFlipRetrace);
+    SDL_NAME(XDGASetViewport) (DGA_Display, DGA_Screen, 0, 0,
+                               XDGAFlipRetrace);
     if (flags & SDL_DOUBLEBUF) {
         if (mode->mode.imageHeight >= (current->h * 2)) {
             current->flags |= SDL_DOUBLEBUF;
@@ -580,16 +580,16 @@
     }
 
     /* Allocate memory tracking for hardware surfaces */
-    DGA_FreeHWSurfaces (this);
+    DGA_FreeHWSurfaces(this);
     if (surfaces_len < 0) {
         surfaces_len = 0;
     }
-    DGA_InitHWSurfaces (this, current, surfaces_mem, surfaces_len);
+    DGA_InitHWSurfaces(this, current, surfaces_mem, surfaces_len);
 
     /* Expose the back buffer as surface memory */
     if (current->flags & SDL_DOUBLEBUF) {
         this->screen = current;
-        DGA_FlipHWSurface (this, current);
+        DGA_FlipHWSurface(this, current);
         this->screen = NULL;
     }
 
@@ -602,7 +602,7 @@
         input_mask = (KeyPressMask | KeyReleaseMask);
         input_mask |= (ButtonPressMask | ButtonReleaseMask);
         input_mask |= PointerMotionMask;
-        SDL_NAME (XDGASelectInput) (DGA_Display, DGA_Screen, input_mask);
+        SDL_NAME(XDGASelectInput) (DGA_Display, DGA_Screen, input_mask);
     }
 
     /* We're done */
@@ -611,38 +611,38 @@
 
 #ifdef DGA_DEBUG
 static void
-DGA_DumpHWSurfaces (_THIS)
+DGA_DumpHWSurfaces(_THIS)
 {
     vidmem_bucket *bucket;
 
-    printf ("Memory left: %d (%d total)\n", surfaces_memleft,
-            surfaces_memtotal);
-    printf ("\n");
-    printf ("         Base  Size\n");
+    printf("Memory left: %d (%d total)\n", surfaces_memleft,
+           surfaces_memtotal);
+    printf("\n");
+    printf("         Base  Size\n");
     for (bucket = &surfaces; bucket; bucket = bucket->next) {
-        printf ("Bucket:  %p, %d (%s)\n", bucket->base, bucket->size,
-                bucket->used ? "used" : "free");
+        printf("Bucket:  %p, %d (%s)\n", bucket->base, bucket->size,
+               bucket->used ? "used" : "free");
         if (bucket->prev) {
             if (bucket->base != bucket->prev->base + bucket->prev->size) {
-                printf ("Warning, corrupt bucket list! (prev)\n");
+                printf("Warning, corrupt bucket list! (prev)\n");
             }
         } else {
             if (bucket != &surfaces) {
-                printf ("Warning, corrupt bucket list! (!prev)\n");
+                printf("Warning, corrupt bucket list! (!prev)\n");
             }
         }
         if (bucket->next) {
             if (bucket->next->base != bucket->base + bucket->size) {
-                printf ("Warning, corrupt bucket list! (next)\n");
+                printf("Warning, corrupt bucket list! (next)\n");
             }
         }
     }
-    printf ("\n");
+    printf("\n");
 }
 #endif
 
 static int
-DGA_InitHWSurfaces (_THIS, SDL_Surface * screen, Uint8 * base, int size)
+DGA_InitHWSurfaces(_THIS, SDL_Surface * screen, Uint8 * base, int size)
 {
     vidmem_bucket *bucket;
 
@@ -650,9 +650,9 @@
     surfaces_memleft = size;
 
     if (surfaces_memleft > 0) {
-        bucket = (vidmem_bucket *) SDL_malloc (sizeof (*bucket));
+        bucket = (vidmem_bucket *) SDL_malloc(sizeof(*bucket));
         if (bucket == NULL) {
-            SDL_OutOfMemory ();
+            SDL_OutOfMemory();
             return (-1);
         }
         bucket->prev = &surfaces;
@@ -675,7 +675,7 @@
     return (0);
 }
 static void
-DGA_FreeHWSurfaces (_THIS)
+DGA_FreeHWSurfaces(_THIS)
 {
     vidmem_bucket *bucket, *freeable;
 
@@ -683,30 +683,30 @@
     while (bucket) {
         freeable = bucket;
         bucket = bucket->next;
-        SDL_free (freeable);
+        SDL_free(freeable);
     }
     surfaces.next = NULL;
 }
 
 static __inline__ void
-DGA_AddBusySurface (SDL_Surface * surface)
+DGA_AddBusySurface(SDL_Surface * surface)
 {
     ((vidmem_bucket *) surface->hwdata)->dirty = 1;
 }
 
 static __inline__ int
-DGA_IsSurfaceBusy (SDL_Surface * surface)
+DGA_IsSurfaceBusy(SDL_Surface * surface)
 {
     return ((vidmem_bucket *) surface->hwdata)->dirty;
 }
 
 static __inline__ void
-DGA_WaitBusySurfaces (_THIS)
+DGA_WaitBusySurfaces(_THIS)
 {
     vidmem_bucket *bucket;
 
     /* Wait for graphic operations to complete */
-    SDL_NAME (XDGASync) (DGA_Display, DGA_Screen);
+    SDL_NAME(XDGASync) (DGA_Display, DGA_Screen);
 
     /* Clear all surface dirty bits */
     for (bucket = &surfaces; bucket; bucket = bucket->next) {
@@ -715,7 +715,7 @@
 }
 
 static int
-DGA_AllocHWSurface (_THIS, SDL_Surface * surface)
+DGA_AllocHWSurface(_THIS, SDL_Surface * surface)
 {
     vidmem_bucket *bucket;
     int size;
@@ -727,19 +727,19 @@
    to be the same.  Others have interesting alignment restrictions.
 */
     if (surface->pitch > SDL_VideoSurface->pitch) {
-        SDL_SetError ("Surface requested wider than screen");
+        SDL_SetError("Surface requested wider than screen");
         return (-1);
     }
     surface->pitch = SDL_VideoSurface->pitch;
     size = surface->h * surface->pitch;
 #ifdef DGA_DEBUG
-    fprintf (stderr, "Allocating bucket of %d bytes\n", size);
+    fprintf(stderr, "Allocating bucket of %d bytes\n", size);
 #endif
-    LOCK_DISPLAY ();
+    LOCK_DISPLAY();
 
     /* Quick check for available mem */
     if (size > surfaces_memleft) {
-        SDL_SetError ("Not enough video memory");
+        SDL_SetError("Not enough video memory");
         retval = -1;
         goto done;
     }
@@ -751,7 +751,7 @@
         }
     }
     if (bucket == NULL) {
-        SDL_SetError ("Video memory too fragmented");
+        SDL_SetError("Video memory too fragmented");
         retval = -1;
         goto done;
     }
@@ -762,11 +762,11 @@
         vidmem_bucket *newbucket;
 
 #ifdef DGA_DEBUG
-        fprintf (stderr, "Adding new free bucket of %d bytes\n", extra);
+        fprintf(stderr, "Adding new free bucket of %d bytes\n", extra);
 #endif
-        newbucket = (vidmem_bucket *) SDL_malloc (sizeof (*newbucket));
+        newbucket = (vidmem_bucket *) SDL_malloc(sizeof(*newbucket));
         if (newbucket == NULL) {
-            SDL_OutOfMemory ();
+            SDL_OutOfMemory();
             retval = -1;
             goto done;
         }
@@ -786,27 +786,26 @@
     bucket->size = size;
     bucket->dirty = 0;
 #ifdef DGA_DEBUG
-    fprintf (stderr, "Allocated %d bytes at %p\n", bucket->size,
-             bucket->base);
+    fprintf(stderr, "Allocated %d bytes at %p\n", bucket->size, bucket->base);
 #endif
     surfaces_memleft -= size;
     surface->flags |= SDL_HWSURFACE;
     surface->pixels = bucket->base;
     surface->hwdata = (struct private_hwdata *) bucket;
   done:
-    UNLOCK_DISPLAY ();
+    UNLOCK_DISPLAY();
     return (retval);
 }
 static void
-DGA_FreeHWSurface (_THIS, SDL_Surface * surface)
+DGA_FreeHWSurface(_THIS, SDL_Surface * surface)
 {
     vidmem_bucket *bucket, *freeable;
 
     /* Wait for any pending operations involving this surface */
-    if (DGA_IsSurfaceBusy (surface)) {
-        LOCK_DISPLAY ();
-        DGA_WaitBusySurfaces (this);
-        UNLOCK_DISPLAY ();
+    if (DGA_IsSurfaceBusy(surface)) {
+        LOCK_DISPLAY();
+        DGA_WaitBusySurfaces(this);
+        UNLOCK_DISPLAY();
     }
 
     /* Look for the bucket in the current list */
@@ -818,7 +817,7 @@
     if (bucket && bucket->used) {
         /* Add the memory back to the total */
 #ifdef DGA_DEBUG
-        printf ("Freeing bucket of %d bytes\n", bucket->size);
+        printf("Freeing bucket of %d bytes\n", bucket->size);
 #endif
         surfaces_memleft += bucket->size;
 
@@ -826,8 +825,8 @@
         bucket->used = 0;
         if (bucket->next && !bucket->next->used) {
 #ifdef DGA_DEBUG
-            printf ("Merging with next bucket, for %d total bytes\n",
-                    bucket->size + bucket->next->size);
+            printf("Merging with next bucket, for %d total bytes\n",
+                   bucket->size + bucket->next->size);
 #endif
             freeable = bucket->next;
             bucket->size += bucket->next->size;
@@ -835,12 +834,12 @@
             if (bucket->next) {
                 bucket->next->prev = bucket;
             }
-            SDL_free (freeable);
+            SDL_free(freeable);
         }
         if (bucket->prev && !bucket->prev->used) {
 #ifdef DGA_DEBUG
-            printf ("Merging with previous bucket, for %d total bytes\n",
-                    bucket->prev->size + bucket->size);
+            printf("Merging with previous bucket, for %d total bytes\n",
+                   bucket->prev->size + bucket->size);
 #endif
             freeable = bucket;
             bucket->prev->size += bucket->size;
@@ -848,7 +847,7 @@
             if (bucket->next) {
                 bucket->next->prev = bucket->prev;
             }
-            SDL_free (freeable);
+            SDL_free(freeable);
         }
     }
     surface->pixels = NULL;
@@ -856,46 +855,46 @@
 }
 
 static __inline__ void
-DGA_dst_to_xy (_THIS, SDL_Surface * dst, int *x, int *y)
+DGA_dst_to_xy(_THIS, SDL_Surface * dst, int *x, int *y)
 {
     *x = (long) ((Uint8 *) dst->pixels - memory_base) % memory_pitch;
     *y = (long) ((Uint8 *) dst->pixels - memory_base) / memory_pitch;
 }
 
 static int
-DGA_FillHWRect (_THIS, SDL_Surface * dst, SDL_Rect * rect, Uint32 color)
+DGA_FillHWRect(_THIS, SDL_Surface * dst, SDL_Rect * rect, Uint32 color)
 {
     int x, y;
     unsigned int w, h;
 
     /* Don't fill the visible part of the screen, wait until flipped */
-    LOCK_DISPLAY ();
+    LOCK_DISPLAY();
     if (was_flipped && (dst == this->screen)) {
-        while (SDL_NAME (XDGAGetViewportStatus) (DGA_Display, DGA_Screen))
+        while (SDL_NAME(XDGAGetViewportStatus) (DGA_Display, DGA_Screen))
             /* Keep waiting for the hardware ... */ ;
         was_flipped = 0;
     }
-    DGA_dst_to_xy (this, dst, &x, &y);
+    DGA_dst_to_xy(this, dst, &x, &y);
     x += rect->x;
     y += rect->y;
     w = rect->w;
     h = rect->h;
 #if 0
-    printf ("Hardware accelerated rectangle fill: %dx%d at %d,%d\n", w, h, x,
-            y);
+    printf("Hardware accelerated rectangle fill: %dx%d at %d,%d\n", w, h, x,
+           y);
 #endif
-    SDL_NAME (XDGAFillRectangle) (DGA_Display, DGA_Screen, x, y, w, h, color);
+    SDL_NAME(XDGAFillRectangle) (DGA_Display, DGA_Screen, x, y, w, h, color);
     if (!(this->screen->flags & SDL_DOUBLEBUF)) {
-        XFlush (DGA_Display);
+        XFlush(DGA_Display);
     }
-    DGA_AddBusySurface (dst);
-    UNLOCK_DISPLAY ();
+    DGA_AddBusySurface(dst);
+    UNLOCK_DISPLAY();
     return (0);
 }
 
 static int
-HWAccelBlit (SDL_Surface * src, SDL_Rect * srcrect,
-             SDL_Surface * dst, SDL_Rect * dstrect)
+HWAccelBlit(SDL_Surface * src, SDL_Rect * srcrect,
+            SDL_Surface * dst, SDL_Rect * dstrect)
 {
     SDL_VideoDevice *this;
     int srcx, srcy;
@@ -904,43 +903,43 @@
 
     this = current_video;
     /* Don't blit to the visible part of the screen, wait until flipped */
-    LOCK_DISPLAY ();
+    LOCK_DISPLAY();
     if (was_flipped && (dst == this->screen)) {
-        while (SDL_NAME (XDGAGetViewportStatus) (DGA_Display, DGA_Screen))
+        while (SDL_NAME(XDGAGetViewportStatus) (DGA_Display, DGA_Screen))
             /* Keep waiting for the hardware ... */ ;
         was_flipped = 0;
     }
-    DGA_dst_to_xy (this, src, &srcx, &srcy);
+    DGA_dst_to_xy(this, src, &srcx, &srcy);
     srcx += srcrect->x;
     srcy += srcrect->y;
-    DGA_dst_to_xy (this, dst, &dstx, &dsty);
+    DGA_dst_to_xy(this, dst, &dstx, &dsty);
     dstx += dstrect->x;
     dsty += dstrect->y;
     w = srcrect->w;
     h = srcrect->h;
 #if 0
-    printf ("Blitting %dx%d from %d,%d to %d,%d\n", w, h, srcx, srcy, dstx,
-            dsty);
+    printf("Blitting %dx%d from %d,%d to %d,%d\n", w, h, srcx, srcy, dstx,
+           dsty);
 #endif
     if ((src->flags & SDL_SRCCOLORKEY) == SDL_SRCCOLORKEY) {
-        SDL_NAME (XDGACopyTransparentArea) (DGA_Display, DGA_Screen,
-                                            srcx, srcy, w, h, dstx, dsty,
-                                            src->format->colorkey);
+        SDL_NAME(XDGACopyTransparentArea) (DGA_Display, DGA_Screen,
+                                           srcx, srcy, w, h, dstx, dsty,
+                                           src->format->colorkey);
     } else {
-        SDL_NAME (XDGACopyArea) (DGA_Display, DGA_Screen,
-                                 srcx, srcy, w, h, dstx, dsty);
+        SDL_NAME(XDGACopyArea) (DGA_Display, DGA_Screen,
+                                srcx, srcy, w, h, dstx, dsty);
     }
     if (!(this->screen->flags & SDL_DOUBLEBUF)) {
-        XFlush (DGA_Display);
+        XFlush(DGA_Display);
     }
-    DGA_AddBusySurface (src);
-    DGA_AddBusySurface (dst);
-    UNLOCK_DISPLAY ();
+    DGA_AddBusySurface(src);
+    DGA_AddBusySurface(dst);
+    UNLOCK_DISPLAY();
     return (0);
 }
 
 static int
-DGA_CheckHWBlit (_THIS, SDL_Surface * src, SDL_Surface * dst)
+DGA_CheckHWBlit(_THIS, SDL_Surface * src, SDL_Surface * dst)
 {
     int accelerated;
 
@@ -968,56 +967,56 @@
 }
 
 static __inline__ void
-DGA_WaitFlip (_THIS)
+DGA_WaitFlip(_THIS)
 {
     if (was_flipped) {
-        while (SDL_NAME (XDGAGetViewportStatus) (DGA_Display, DGA_Screen))
+        while (SDL_NAME(XDGAGetViewportStatus) (DGA_Display, DGA_Screen))
             /* Keep waiting for the hardware ... */ ;
         was_flipped = 0;
     }
 }
 
 static int
-DGA_LockHWSurface (_THIS, SDL_Surface * surface)
+DGA_LockHWSurface(_THIS, SDL_Surface * surface)
 {
     if (surface == this->screen) {
-        SDL_mutexP (hw_lock);
-        LOCK_DISPLAY ();
-        if (DGA_IsSurfaceBusy (surface)) {
-            DGA_WaitBusySurfaces (this);
+        SDL_mutexP(hw_lock);
+        LOCK_DISPLAY();
+        if (DGA_IsSurfaceBusy(surface)) {
+            DGA_WaitBusySurfaces(this);
         }
-        DGA_WaitFlip (this);
-        UNLOCK_DISPLAY ();
+        DGA_WaitFlip(this);
+        UNLOCK_DISPLAY();
     } else {
-        if (DGA_IsSurfaceBusy (surface)) {
-            LOCK_DISPLAY ();
-            DGA_WaitBusySurfaces (this);
-            UNLOCK_DISPLAY ();
+        if (DGA_IsSurfaceBusy(surface)) {
+            LOCK_DISPLAY();
+            DGA_WaitBusySurfaces(this);
+            UNLOCK_DISPLAY();
         }
     }
     return (0);
 }
 static void
-DGA_UnlockHWSurface (_THIS, SDL_Surface * surface)
+DGA_UnlockHWSurface(_THIS, SDL_Surface * surface)
 {
     if (surface == this->screen) {
-        SDL_mutexV (hw_lock);
+        SDL_mutexV(hw_lock);
     }
 }
 
 static int
-DGA_FlipHWSurface (_THIS, SDL_Surface * surface)
+DGA_FlipHWSurface(_THIS, SDL_Surface * surface)
 {
     /* Wait for vertical retrace and then flip display */
-    LOCK_DISPLAY ();
-    if (DGA_IsSurfaceBusy (this->screen)) {
-        DGA_WaitBusySurfaces (this);
+    LOCK_DISPLAY();
+    if (DGA_IsSurfaceBusy(this->screen)) {
+        DGA_WaitBusySurfaces(this);
     }
-    DGA_WaitFlip (this);
-    SDL_NAME (XDGASetViewport) (DGA_Display, DGA_Screen,
-                                0, flip_yoffset[flip_page], XDGAFlipRetrace);
-    XFlush (DGA_Display);
-    UNLOCK_DISPLAY ();
+    DGA_WaitFlip(this);
+    SDL_NAME(XDGASetViewport) (DGA_Display, DGA_Screen,
+                               0, flip_yoffset[flip_page], XDGAFlipRetrace);
+    XFlush(DGA_Display);
+    UNLOCK_DISPLAY();
     was_flipped = 1;
     flip_page = !flip_page;
 
@@ -1026,14 +1025,14 @@
 }
 
 static void
-DGA_DirectUpdate (_THIS, int numrects, SDL_Rect * rects)
+DGA_DirectUpdate(_THIS, int numrects, SDL_Rect * rects)
 {
     /* The application is already updating the visible video memory */
     return;
 }
 
 static int
-DGA_SetColors (_THIS, int firstcolor, int ncolors, SDL_Color * colors)
+DGA_SetColors(_THIS, int firstcolor, int ncolors, SDL_Color * colors)
 {
     int i;
     XColor *xcmap;
@@ -1042,7 +1041,7 @@
     if (!DGA_colormap) {
         return (0);
     }
-    xcmap = SDL_stack_alloc (XColor, ncolors);
+    xcmap = SDL_stack_alloc(XColor, ncolors);
     for (i = 0; i < ncolors; ++i) {
         xcmap[i].pixel = firstcolor + i;
         xcmap[i].red = (colors[i].r << 8) | colors[i].r;
@@ -1050,25 +1049,25 @@
         xcmap[i].blue = (colors[i].b << 8) | colors[i].b;
         xcmap[i].flags = (DoRed | DoGreen | DoBlue);
     }
-    LOCK_DISPLAY ();
-    XStoreColors (DGA_Display, DGA_colormap, xcmap, ncolors);
-    XSync (DGA_Display, False);
-    UNLOCK_DISPLAY ();
-    SDL_stack_free (xcmap);
+    LOCK_DISPLAY();
+    XStoreColors(DGA_Display, DGA_colormap, xcmap, ncolors);
+    XSync(DGA_Display, False);
+    UNLOCK_DISPLAY();
+    SDL_stack_free(xcmap);
 
     /* That was easy. :) */
     return (1);
 }
 
 int
-DGA_SetGammaRamp (_THIS, Uint16 * ramp)
+DGA_SetGammaRamp(_THIS, Uint16 * ramp)
 {
     int i, ncolors;
     XColor xcmap[256];
 
     /* See if actually setting the gamma is supported */
     if (DGA_visualClass != DirectColor) {
-        SDL_SetError ("Gamma correction not supported on this visual");
+        SDL_SetError("Gamma correction not supported on this visual");
         return (-1);
     }
 
@@ -1080,47 +1079,47 @@
     }
     for (i = 0; i < ncolors; ++i) {
         Uint8 c = (256 * i / ncolors);
-        xcmap[i].pixel = SDL_MapRGB (this->screen->format, c, c, c);
+        xcmap[i].pixel = SDL_MapRGB(this->screen->format, c, c, c);
         xcmap[i].red = ramp[0 * 256 + c];
         xcmap[i].green = ramp[1 * 256 + c];
         xcmap[i].blue = ramp[2 * 256 + c];
         xcmap[i].flags = (DoRed | DoGreen | DoBlue);
     }
-    LOCK_DISPLAY ();
-    XStoreColors (DGA_Display, DGA_colormap, xcmap, ncolors);
-    XSync (DGA_Display, False);
-    UNLOCK_DISPLAY ();
+    LOCK_DISPLAY();
+    XStoreColors(DGA_Display, DGA_colormap, xcmap, ncolors);
+    XSync(DGA_Display, False);
+    UNLOCK_DISPLAY();
     return (0);
 }
 
 void
-DGA_VideoQuit (_THIS)
+DGA_VideoQuit(_THIS)
 {
     int i, j;
 
     if (DGA_Display) {
         /* Free colormap, if necessary */
         if (DGA_colormap) {
-            XFreeColormap (DGA_Display, DGA_colormap);
+            XFreeColormap(DGA_Display, DGA_colormap);
             DGA_colormap = 0;
         }
 
         /* Unmap memory and reset video mode */
-        SDL_NAME (XDGACloseFramebuffer) (DGA_Display, DGA_Screen);
+        SDL_NAME(XDGACloseFramebuffer) (DGA_Display, DGA_Screen);
         if (this->screen) {
             /* Tell SDL not to free the pixels */
-            DGA_FreeHWSurface (this, this->screen);
+            DGA_FreeHWSurface(this, this->screen);
         }
-        SDL_NAME (XDGASetMode) (DGA_Display, DGA_Screen, 0);
+        SDL_NAME(XDGASetMode) (DGA_Display, DGA_Screen, 0);
 
         /* Clear the lock mutex */
         if (hw_lock != NULL) {
-            SDL_DestroyMutex (hw_lock);
+            SDL_DestroyMutex(hw_lock);
             hw_lock = NULL;
         }
 #ifdef LOCK_DGA_DISPLAY
         if (event_lock != NULL) {
-            SDL_DestroyMutex (event_lock);
+            SDL_DestroyMutex(event_lock);
             event_lock = NULL;
         }
 #endif /* LOCK_DGA_DISPLAY */
@@ -1129,22 +1128,22 @@
         for (i = 0; i < NUM_MODELISTS; ++i) {
             if (SDL_modelist[i] != NULL) {
                 for (j = 0; SDL_modelist[i][j]; ++j) {
-                    SDL_free (SDL_modelist[i][j]);
+                    SDL_free(SDL_modelist[i][j]);
                 }
-                SDL_free (SDL_modelist[i]);
+                SDL_free(SDL_modelist[i]);
                 SDL_modelist[i] = NULL;
             }
         }
 
         /* Clean up the memory bucket list */
-        DGA_FreeHWSurfaces (this);
+        DGA_FreeHWSurfaces(this);
 
         /* Restore DPMS and screensaver settings */
-        X11_RestoreScreenSaver (DGA_Display, screensaver_timeout,
-                                dpms_enabled);
+        X11_RestoreScreenSaver(DGA_Display, screensaver_timeout,
+                               dpms_enabled);
 
         /* Close up the display */
-        XCloseDisplay (DGA_Display);
+        XCloseDisplay(DGA_Display);
     }
 }