actually changed my mind for a bit

libs/x11/inc.c

@@ -7,4 +7,3 @@
 #include "client.h"
 #include "client.c"
 #include "init.c"
-#include "mod.c"

libs/x11/inc.h

@@ -2,7 +2,6 @@
 #include "lib.h"
 
 #include "init.h"
-#include "mod.h"
 #include "xrdb.h"
 #include "xim.h"
 #include "key.h"

libs/x11/init.c

@@ -138,3 +138,12 @@ void setupdisplay(void) {
     match();
     drawmenu();
 }
+
+Display * opendisplay(char *disp) {
+    return XOpenDisplay(disp);
+}
+
+void set_screen(Display *disp) {
+    screen = DefaultScreen(disp);
+    root = RootWindow(disp, screen);
+}

libs/x11/init.h

@@ -5,3 +5,5 @@ static Display *dpy;
 static Window root, parentwin, win;
 
 static void setupdisplay(void);
+static void set_screen(Display *disp);
+static Display * opendisplay(char *disp);

libs/x11/mod.c

@@ -1,17 +0,0 @@
-/* See LICENSE file for copyright and license details. */
-
-int get_locale(char *modifier_list) {
-    if (XSetLocaleModifiers(modifier_list)) {
-        return 0;
-    } else {
-        return 1;
-    }
-}
-
-int supports_locale() {
-    if (XSupportsLocale()) {
-        return 0;
-    } else {
-        return 1;
-    }
-}

libs/x11/mod.h

@@ -1,4 +0,0 @@
-/* See LICENSE file for copyright and license details. */
-
-static int get_locale(char *modifier_list);
-static int supports_locale();

spmenu.c

@@ -534,26 +534,25 @@ int main(int argc, char *argv[]) {
         strcpy(modetext, normtext);
     }
 
-    if (!setlocale(LC_CTYPE, "") || !supports_locale())
+    if (!setlocale(LC_CTYPE, "") || !XSupportsLocale())
         fputs("warning: no locale support\n", stderr); // invalid locale, so notify the user about it
 
-    if (!get_locale("")) {
+    if (!XSetLocaleModifiers(""))
         fputs("warning: no locale modifiers support\n", stderr);
-    }
 
-    if (!(dpy = XOpenDisplay(NULL)))
+    if (!(dpy = opendisplay(NULL)))
         die("spmenu: cannot open display"); // failed to open display
 
     // set screen and root window
-    screen = DefaultScreen(dpy);
-    root = RootWindow(dpy, screen);
+    set_screen(dpy);
 
     // parent window is the root window (ie. window manager) because we're not embedding
     if (!embed || !(parentwin = strtol(embed, NULL, 0)))
         parentwin = root;
 
-    if (!XGetWindowAttributes(dpy, parentwin, &wa))
+    if (!XGetWindowAttributes(dpy, parentwin, &wa)) {
         die("spmenu: could not get embedding window attributes: 0x%lx", parentwin);
+    }
 
     xinitvisual(); // init visual and create drawable after
     drw = drw_create(dpy, screen, root, wa.width, wa.height, visual, depth, cmap); // wrapper function creating a drawable