added comments and fixed import string

src/input.h

@@ -1,6 +1,6 @@
 #pragma once 
 
-#include "command.h "
+#include "command.h"
 
 typedef enum {
     BUTTON_UP,

src/main.c

@@ -1,7 +1,6 @@
 #include <stdio.h>
 #include <string.h>
 
-#include "command.h "
 #include "command.h"
 #include "input.h"
 
@@ -11,19 +10,21 @@ int printButton(const char* string) {
     return printf(string);
 }
 
+// this should live in the app logic
 void commandWrapper(void* args, void* ret) {
     // we cast ret to a int and then place the return val in to it
     *(int*)(ret) = printButton(*(const char**)(args));
 }
 
 
+// this should be in your input driver, so in here you may have a task
+// that is getting these inputs and adding them to a queue
 static Inputs_e current_input = BUTTON_DOWN;
 Inputs_e getinput() {
     return current_input;
 }
 
 
-// this is the only part i dont like
 const char* button_up_cmd_args = "up button pressed\n";
 int button_up_cmd_return_val = 0; // dont leave thing uninstalised
 
@@ -36,6 +37,7 @@ int button_right_cmd_return_val = 0; // dont leave thing uninstalised
 const char* button_down_cmd_args = "down button pressed\n";
 int button_down_cmd_return_val = 0; // dont leave thing uninstalised
 
+// this should be implemented with your app logic
 static Input_t input_device = (Input_t){
     .button_up_cmd = (Command_t){
         .args = (void*)(&button_up_cmd_args),