Remove a few useless header inclusion via deheader

Makefile

@@ -13,7 +13,7 @@ BASHCPL = $(PREFIX)/share/bash-completion/completions
 ZSHCPL = $(PREFIX)/share/zsh/site-functions
 
 WM_SRC = bspwm.c helpers.c settings.c monitor.c desktop.c tree.c history.c \
-	 events.c window.c messages.c query.c restore.c rules.c ewmh.c
+	 events.c window.c messages.c query.c restore.c rule.c ewmh.c
 WM_OBJ = $(WM_SRC:.c=.o)
 CL_SRC = bspc.c helpers.c
 CL_OBJ = $(CL_SRC:.c=.o)

Sourcedeps

@@ -1,15 +1,15 @@
 bspc.o: bspc.c common.h helpers.h
-bspwm.o: bspwm.c bspwm.h common.h desktop.h events.h ewmh.h helpers.h messages.h monitor.h rules.h settings.h tree.h types.h window.h
+bspwm.o: bspwm.c bspwm.h common.h desktop.h events.h ewmh.h helpers.h messages.h monitor.h rule.h settings.h tree.h types.h window.h
 desktop.o: desktop.c bspwm.h desktop.h ewmh.h helpers.h history.h tree.h types.h window.h
-events.o: events.c bspwm.h events.h ewmh.h helpers.h monitor.h query.h rules.h settings.h tree.h types.h window.h
+events.o: events.c bspwm.h events.h ewmh.h helpers.h monitor.h query.h settings.h tree.h types.h window.h
 ewmh.o: ewmh.c bspwm.h ewmh.h helpers.h settings.h tree.h types.h
 helpers.o: helpers.c bspwm.h helpers.h types.h
-history.o: history.c helpers.h history.h query.h types.h
+history.o: history.c helpers.h query.h types.h
-messages.o: messages.c bspwm.h common.h desktop.h events.h ewmh.h helpers.h messages.h monitor.h query.h restore.h rules.h settings.h tree.h types.h window.h
+messages.o: messages.c bspwm.h desktop.h events.h ewmh.h helpers.h messages.h monitor.h query.h restore.h rule.h settings.h tree.h types.h window.h
 monitor.o: monitor.c bspwm.h desktop.h ewmh.h helpers.h monitor.h tree.h types.h window.h
-query.o: query.c bspwm.h helpers.h history.h messages.h query.h settings.h tree.h types.h
+query.o: query.c bspwm.h helpers.h history.h messages.h query.h tree.h types.h
 restore.o: restore.c bspwm.h desktop.h ewmh.h helpers.h history.h monitor.h query.h restore.h settings.h tree.h types.h
-rules.o: rules.c bspwm.h ewmh.h helpers.h query.h rules.h types.h window.h
+rule.o: rule.c bspwm.h ewmh.h helpers.h query.h rule.h types.h window.h
-settings.o: settings.c bspwm.h common.h helpers.h settings.h types.h
+settings.o: settings.c bspwm.h helpers.h settings.h types.h
 tree.o: tree.c bspwm.h desktop.h ewmh.h helpers.h history.h query.h settings.h tree.h types.h window.h
-window.o: window.c bspwm.h ewmh.h helpers.h query.h rules.h settings.h tree.h types.h window.h
+window.o: window.c bspwm.h ewmh.h helpers.h query.h rule.h settings.h tree.h types.h window.h

bspwm.c

@@ -2,28 +2,22 @@
 #include <stdio.h>
 #include <string.h>
 #include <unistd.h>
-#include <signal.h>
 #include <fcntl.h>
 #include <ctype.h>
 #include <sys/socket.h>
 #include <sys/un.h>
 #include <sys/select.h>
-#include <xcb/xcb.h>
-#include <xcb/xcb_event.h>
-#include <xcb/xcb_ewmh.h>
-#include <xcb/randr.h>
 #include "types.h"
 #include "desktop.h"
 #include "monitor.h"
 #include "settings.h"
 #include "messages.h"
-#include "rules.h"
 #include "events.h"
 #include "common.h"
-#include "helpers.h"
-#include "window.h"
 #include "bspwm.h"
 #include "tree.h"
+#include "window.h"
+#include "rule.h"
 #include "ewmh.h"
 
 int main(int argc, char *argv[])

desktop.c

@@ -1,10 +1,9 @@
 #include <stdlib.h>
 #include <string.h>
 #include "bspwm.h"
-#include "types.h"
 #include "desktop.h"
-#include "history.h"
 #include "tree.h"
+#include "history.h"
 #include "window.h"
 #include "ewmh.h"
 

events.c

@@ -1,20 +1,13 @@
-#include <stdio.h>
 #include <stdlib.h>
-#include <string.h>
-#include <xcb/xcb.h>
-#include <xcb/randr.h>
-#include <xcb/xcb_event.h>
 #include <xcb/xcb_icccm.h>
 #include "types.h"
-#include "monitor.h"
 #include "bspwm.h"
 #include "settings.h"
-#include "helpers.h"
-#include "window.h"
 #include "events.h"
-#include "tree.h"
+#include "monitor.h"
+#include "window.h"
 #include "query.h"
-#include "rules.h"
+#include "tree.h"
 #include "ewmh.h"
 
 void handle_event(xcb_generic_event_t *evt)

ewmh.c

@@ -1,8 +1,5 @@
-#include <stdio.h>
 #include <string.h>
 #include <unistd.h>
-#include <xcb/xcb_ewmh.h>
-#include "types.h"
 #include "bspwm.h"
 #include "settings.h"
 #include "tree.h"

helpers.c

@@ -1,10 +1,7 @@
 #include <stdlib.h>
 #include <string.h>
 #include <math.h>
-#include <xcb/xcb.h>
-#include <xcb/xcb_event.h>
 #include "bspwm.h"
-#include "helpers.h"
 
 void warn(char *fmt, ...)
 {

history.c

@@ -1,7 +1,6 @@
 #include <stdlib.h>
 #include "types.h"
 #include "query.h"
-#include "history.h"
 
 focus_history_t *make_focus_history(void)
 {

messages.c

@@ -3,20 +3,17 @@
 #include <stdlib.h>
 #include <errno.h>
 #include "settings.h"
-#include "messages.h"
 #include "query.h"
-#include "restore.h"
+#include "bspwm.h"
-#include "common.h"
+#include "tree.h"
-#include "types.h"
 #include "desktop.h"
 #include "monitor.h"
-#include "bspwm.h"
-#include "ewmh.h"
-#include "helpers.h"
 #include "window.h"
+#include "rule.h"
+#include "restore.h"
 #include "events.h"
-#include "tree.h"
+#include "ewmh.h"
-#include "rules.h"
+#include "messages.h"
 
 bool cmd_window(char **args, int num)
 {

monitor.c

@@ -1,10 +1,9 @@
 #include <stdlib.h>
 #include <string.h>
 #include "bspwm.h"
-#include "types.h"
-#include "desktop.h"
 #include "tree.h"
 #include "monitor.h"
+#include "desktop.h"
 #include "window.h"
 #include "ewmh.h"
 

query.c

@@ -1,11 +1,9 @@
-#include <stdlib.h>
 #include <string.h>
 #include <stdio.h>
 #include "bspwm.h"
 #include "tree.h"
-#include "history.h"
-#include "settings.h"
 #include "messages.h"
+#include "history.h"
 #include "query.h"
 
 void query_monitors(coordinates_t loc, domain_t dom, char *rsp)

restore.c

@@ -1,14 +1,14 @@
 #include <ctype.h>
 #include <string.h>
+#include "bspwm.h"
 #include "types.h"
-#include "desktop.h"
 #include "monitor.h"
-#include "history.h"
+#include "desktop.h"
 #include "tree.h"
 #include "settings.h"
-#include "ewmh.h"
-#include "bspwm.h"
 #include "query.h"
+#include "history.h"
+#include "ewmh.h"
 #include "restore.h"
 
 void restore_tree(char *file_path)

rule.c

@@ -1,13 +1,11 @@
 #include <stdio.h>
 #include <string.h>
 #include <xcb/xcb_icccm.h>
-#include <xcb/xcb_ewmh.h>
-#include "window.h"
-#include "types.h"
 #include "bspwm.h"
 #include "ewmh.h"
-#include "rules.h"
+#include "window.h"
 #include "query.h"
+#include "rule.h"
 
 rule_t *make_rule(void)
 {

rule.h

@@ -1,5 +1,5 @@
-#ifndef _RULES_H
+#ifndef _RULE_H
-#define _RULES_H
+#define _RULE_H
 
 #define MATCH_ALL  "*"
 

settings.c

@@ -1,13 +1,8 @@
-#include <stdio.h>
 #include <stdlib.h>
 #include <string.h>
 #include <unistd.h>
 #include <sys/wait.h>
-#include <xcb/xcb.h>
-#include <xcb/xcb_event.h>
 #include "bspwm.h"
-#include "helpers.h"
-#include "common.h"
 #include "settings.h"
 
 void run_config(void)

tree.c

@@ -3,15 +3,13 @@
 #include <limits.h>
 #include <float.h>
 #include "settings.h"
-#include "helpers.h"
 #include "window.h"
-#include "types.h"
-#include "desktop.h"
-#include "history.h"
-#include "query.h"
 #include "bspwm.h"
 #include "ewmh.h"
 #include "tree.h"
+#include "desktop.h"
+#include "history.h"
+#include "query.h"
 
 node_t *make_node(void)
 {

window.c

@@ -1,17 +1,12 @@
-#include <stdio.h>
 #include <stdlib.h>
-#include <stdarg.h>
 #include <string.h>
-#include <xcb/xcb.h>
-#include <xcb/xcb_event.h>
-#include <xcb/xcb_icccm.h>
 #include "types.h"
 #include "tree.h"
 #include "bspwm.h"
 #include "settings.h"
 #include "ewmh.h"
-#include "rules.h"
 #include "query.h"
+#include "rule.h"
 #include "window.h"
 
 pointer_state_t *make_pointer_state(void)