Move keypress print func to before main

Means we need no separate declaration, and it's a lot easier to infer
that the function is pure.

1 files changed, 12 insertions, 14 deletions

diff --git a/xkbcat.c b/xkbcat.c
index 4fd31a5..7a39a99 100644
--- a/xkbcat.c
+++ b/xkbcat.c
@@ -31,7 +31,18 @@ USAGE: xkbcat [-display <display>] [-delay <nanosec>] [-up] [-time]\n\
     exit(0);
 }
 
-void printKeyPress(Display * disp, int code, bool down, bool printKeyUps, long time, bool printTimes);
+// Since `XKeysymToString` returns a string of unknown length that shouldn't be
+// modified, so it makes more sense to just `printf` it in-place.
+void printKeyPress(Display * disp, int code, bool down, bool printKeyUps, long timestamp, bool printTimes) {
+
+    KeySym s = XkbKeycodeToKeysym(disp, code, 0, 0); if (NoSymbol == s) return;
+    char * str = XKeysymToString(s);                 if (NULL == str)   return;
+
+    if (printKeyUps) printf("%s ", (down ? "+" : "-"));
+    printf("%s", str);
+    if (printTimes)  printf(" %ld", timestamp);
+    printf("\n");
+}
 
 int main(int argc, char * argv[]) {
 
@@ -89,16 +100,3 @@ int main(int argc, char * argv[]) {
         nanosleep(&sleepTime, NULL);
     }
 }
-
-// Since `XKeysymToString` returns a string of unknown length that shouldn't be
-// modified, so it makes more sense to just `printf` it in-place.
-void printKeyPress(Display * disp, int code, bool down, bool printKeyUps, long timestamp, bool printTimes) {
-
-    KeySym s = XkbKeycodeToKeysym(disp, code, 0, 0); if (NoSymbol == s) return;
-    char * str = XKeysymToString(s);                 if (NULL == str)   return;
-
-    if (printKeyUps) printf("%s ", (down ? "+" : "-"));
-    printf("%s", str);
-    if (printTimes)  printf(" %ld", timestamp);
-    printf("\n");
-}