Run fix_include checker on all source files (#372)

ci-build.py

@@ -66,7 +66,7 @@
 
 if args.iwyu:
     # Check lexicographical order of #include directives (cheap pre-check)
-    sp.check_call('fix_include --dry_run --sort_only --reorder /hlwm/src/*.h', shell=True)
+    sp.check_call('fix_include --dry_run --sort_only --reorder /hlwm/*/*.{h,cpp,c}', shell=True, executable='bash')
 
     # Run include-what-you-use (just printing the result for now)
     sp.check_call(f'iwyu_tool -p . -j "$(nproc)" -- --mapping_file=/hlwm/.hlwm.imp', shell=True, cwd=build_dir)

ipc-client/ipc-client.c

@@ -3,18 +3,18 @@
  * This software is licensed under the "Simplified BSD License".
  * See LICENSE for details */
 
-#include <stdio.h>
-#include <string.h>
-#include <stdlib.h>
-#include <stdbool.h>
 #include <X11/X.h>
 #include <X11/Xatom.h>
 #include <X11/Xlib.h>
 #include <X11/Xutil.h>
+#include <stdbool.h>
+#include <stdio.h>
+#include <stdlib.h>
+#include <string.h>
 
 #include "../src/ipc-protocol.h"
-#include "ipc-client.h"
 #include "client-utils.h"
+#include "ipc-client.h"
 
 struct HCConnection {
     Display*    display;

ipc-client/main.c

@@ -4,18 +4,18 @@
  * See LICENSE for details */
 
 #include <X11/Xlib.h>
-#include <stdio.h>
-#include <stdlib.h>
-#include <stdbool.h>
+#include <assert.h>
 #include <getopt.h>
-#include <signal.h>
 #include <regex.h>
-#include <assert.h>
+#include <signal.h>
+#include <stdbool.h>
+#include <stdio.h>
+#include <stdlib.h>
 #include <string.h>
 
 #include "../src/ipc-protocol.h"
-#include "ipc-client.h"
 #include "client-utils.h"
+#include "ipc-client.h"
 
 static void print_help(char* command, FILE* file);
 static void init_hook_regex(int argc, char* argv[]);

src/frametree.cpp

@@ -1,14 +1,14 @@
 #include "frametree.h"
 
+#include <algorithm>
+
 #include "client.h"
 #include "ipc-protocol.h"
 #include "layout.h"
 #include "monitor.h"
 #include "tag.h"
 #include "utils.h"
 
-#include <algorithm>
-
 using std::endl;
 using std::function;
 using std::make_shared;

src/hook.cpp

@@ -4,9 +4,9 @@
 #include <X11/Xatom.h>
 #include <X11/Xlib.h>
 #include <X11/Xutil.h>
+#include <stdarg.h>
 #include <cassert>
 #include <cstdio>
-#include <stdarg.h>
 
 #include "glib-backports.h"
 #include "globals.h"

src/main.cpp

@@ -1,11 +1,11 @@
+#include <X11/X.h>
+#include <X11/Xatom.h>
+#include <X11/Xlib.h>
 #include <errno.h>
 #include <getopt.h>
 #include <sys/select.h>
 #include <sys/wait.h>
 #include <unistd.h>
-#include <X11/X.h>
-#include <X11/Xatom.h>
-#include <X11/Xlib.h>
 #include <cassert>
 #include <csignal>
 #include <cstdio>

src/rules.cpp

@@ -5,11 +5,11 @@
 #include <algorithm>
 #include <cstdio>
 
-#include "globals.h"
-#include "utils.h"
-#include "ewmh.h"
 #include "client.h"
+#include "ewmh.h"
+#include "globals.h"
 #include "hook.h"
+#include "utils.h"
 
 using std::string;
 

src/tag.cpp

@@ -4,10 +4,10 @@
 
 #include "client.h"
 #include "clientmanager.h"
+#include "frametree.h"
 #include "globals.h"
 #include "hook.h"
 #include "ipc-protocol.h"
-#include "frametree.h"
 #include "layout.h"
 #include "monitor.h"
 #include "root.h"

utilities/XFetchName.c

@@ -1,9 +1,9 @@
-#include <stdio.h>
-#include <stdlib.h>
+#include <X11/Xatom.h>
 #include <X11/Xlib.h>
 #include <X11/Xproto.h>
 #include <X11/Xutil.h>
-#include <X11/Xatom.h>
+#include <stdio.h>
+#include <stdlib.h>
 
 #define DIE(...) do { fprintf(stderr, __VA_ARGS__); exit(1); } while(0)