From 68098f44db94d7cd3281d9ae2d34cd7de0a46441 Mon Sep 17 00:00:00 2001
From: Camilla Berglund <elmindreda@elmindreda.org>
Date: Fri, 4 Jan 2013 07:11:30 +0100
Subject: [PATCH] Cleanup.

---
 src/x11_init.c   | 9 ++++-----
 src/x11_window.c | 6 +++---
 2 files changed, 7 insertions(+), 8 deletions(-)

diff --git a/src/x11_init.c b/src/x11_init.c
index 8fa155bb..72fc90a8 100644
--- a/src/x11_init.c
+++ b/src/x11_init.c
@@ -395,7 +395,7 @@ static Atom getSupportedAtom(Atom* supportedAtoms,
 // Check whether the running window manager is EWMH-compliant
 //========================================================================
 
-static void initEWMH(void)
+static void detectEWMH(void)
 {
     Window* windowFromRoot = NULL;
     Window* windowFromChild = NULL;
@@ -549,6 +549,9 @@ static GLboolean initDisplay(void)
     // the keyboard mapping.
     updateKeyCodeLUT();
 
+    // Detect whether an EWMH-conformant window manager is running
+    detectEWMH();
+
     // Find or create selection property atom
     _glfw.x11.selection.property =
         XInternAtom(_glfw.x11.display, "GLFW_SELECTION", False);
@@ -638,14 +641,11 @@ int _glfwPlatformInit(void)
     if (!_glfwInitOpenGL())
         return GL_FALSE;
 
-    initEWMH();
-
     _glfw.x11.cursor = createNULLCursor();
 
     if (!_glfwInitJoysticks())
         return GL_FALSE;
 
-    // Start the timer
     _glfwInitTimer();
 
     return GL_TRUE;
@@ -672,7 +672,6 @@ void _glfwPlatformTerminate(void)
 
     terminateDisplay();
 
-    // Free clipboard memory
     if (_glfw.x11.selection.string)
         free(_glfw.x11.selection.string);
 }
diff --git a/src/x11_window.c b/src/x11_window.c
index e328f3b5..753a826a 100644
--- a/src/x11_window.c
+++ b/src/x11_window.c
@@ -109,9 +109,9 @@ static GLboolean createWindow(_GLFWwindow* window,
 
         if (wndconfig->monitor == NULL)
         {
-            // The /only/ reason for setting the background pixel here is that
-            // otherwise our window won't get any decorations on systems using
-            // certain versions of Compiz on Intel hardware
+            // HACK: This is a workaround for windows without a background pixel
+            // not getting any decorations on certain older versions of Compiz
+            // running on Intel hardware
             wa.background_pixel = BlackPixel(_glfw.x11.display,
                                              _glfw.x11.screen);
             wamask |= CWBackPixel;