restructed all the files + uci done now

also broke the symbol analyser in the process, i didn't help much, i had
to pull out the pen & paper, but it made a cool graph never the less

multithreading was a pain! but it got it in the end

fixed the structs naming convention, if it's used outside by other
files, it's typedeffed, otherwise it's not

fixed the recursive imports issued i had in go_args*, where i was
casting it to void* in uci to make it work

now response.c doesn't deal with ipc, really happy with that change
now every single thing has its own responsibility

5 files changed, 89 insertions, 78 deletions

diff --git a/include/uci/command.h b/include/uci/command.h
deleted file mode 100644
index ba9c51d..0000000
--- a/include/uci/command.h
+++ /dev/null
@@ -1,21 +0,0 @@
-#ifndef UCI_COMMAND_H
-#define UCI_COMMAND_H
-
-#define MAX_TOKEN_SIZE 32
-
-typedef struct {
-  char root[MAX_TOKEN_SIZE];
-  int args_count;
-  int args_capacity;
-  char* args;
-  bool empty;
-} ucicmd;
-
-ucicmd ucicmd_init();
-void ucicmd_add(ucicmd* cmd, const char* token);
-void ucicmd_set_root(ucicmd* cmd, const char* token);
-void ucicmd_append_arg(ucicmd* cmd, const char* token);
-char* ucicmd_get_arg(ucicmd cmd, int i);
-void ucicmd_deinit(ucicmd cmd);
-
-#endif // UCI_COMMAND_H
diff --git a/include/uci/internals/command.h b/include/uci/internals/command.h
new file mode 100644
index 0000000..2994b04
--- /dev/null
+++ b/include/uci/internals/command.h
@@ -0,0 +1,21 @@
+#ifndef UCI_COMMAND_H
+#define UCI_COMMAND_H
+
+#define MAX_TOKEN_SIZE 32
+
+typedef struct {
+  char root[MAX_TOKEN_SIZE];
+  int args_count;
+  int args_capacity;
+  char* args;
+  bool empty;
+} uci_cmd_t;
+
+uci_cmd_t uci_cmd_init();
+void uci_cmd_add(uci_cmd_t *cmd, const char *token);
+void uci_cmd_set_root(uci_cmd_t *cmd, const char *token);
+void uci_cmd_append_arg(uci_cmd_t *cmd, const char *token);
+char *uci_cmd_get_arg(uci_cmd_t cmd, int i);
+void uci_cmd_deinit(uci_cmd_t cmd);
+
+#endif // UCI_COMMAND_H
diff --git a/include/uci/internals/response.h b/include/uci/internals/response.h
new file mode 100644
index 0000000..a0a6e99
--- /dev/null
+++ b/include/uci/internals/response.h
@@ -0,0 +1,24 @@
+#ifndef UCI_RESPONSE_H
+#define UCI_RESPONSE_H
+
+#include "uci/internals/command.h"
+#include "uci/state.h"
+
+enum {
+  UCI_STATE_INITIAL,
+  UCI_STATE_IDLE,
+  UCI_STATE_SYNC,
+  UCI_STATE_PING,
+  UCI_STATE_ACTIVE,
+  UCI_STATE_HALT,
+};
+
+void handle_uci(uci_state_t *state, uci_cmd_t cmd);
+void handle_initial(uci_state_t *state, uci_cmd_t cmd);
+void handle_idle(uci_state_t *state, uci_cmd_t cmd);
+void handle_sync(uci_state_t *state, uci_cmd_t cmd);
+void handle_ping(uci_state_t *state, uci_cmd_t cmd);
+void handle_active(uci_state_t *state, uci_cmd_t cmd);
+void handle_halt(uci_state_t *state, uci_cmd_t cmd);
+
+#endif // UCI_RESPONSE_H
diff --git a/include/uci/response.h b/include/uci/response.h
deleted file mode 100644
index 320b1f0..0000000
--- a/include/uci/response.h
+++ /dev/null
@@ -1,24 +0,0 @@
-#ifndef RESPONSE_H
-#define RESPONSE_H
-
-#include "command.h"
-#include "state.h"
-#include "../ipc.h"
-
-void handle_uci(
-  uci_state *state,
-  engine_messages* engine_message,
-  ucicmd cmd
-);
-void handle_initial(uci_state *state, ucicmd cmd);
-void handle_idle(uci_state *state, ucicmd cmd);
-void handle_sync(uci_state *state, ucicmd cmd);
-void handle_ping(uci_state *state, ucicmd cmd);
-void handle_active(
-  uci_state *state,
-  engine_messages* engine_message,
-  ucicmd cmd
-);
-void handle_halt(uci_state *state, ucicmd cmd);
-
-#endif // RESPONSE_H
diff --git a/include/uci/state.h b/include/uci/state.h
index a1df891..51207f3 100644
--- a/include/uci/state.h
+++ b/include/uci/state.h
@@ -1,11 +1,12 @@
 #ifndef UCI_STATE_H
 #define UCI_STATE_H
 
-#include <stdatomic.h>
-#include "../fen.h"
+#include "ipc/go_args.h"
+#include "ipc/moves.h"
+#include "bitboard.h"
 
 typedef struct {
-  char* data;
+  char *data;
   int length;
 } str_t;
 
@@ -19,7 +20,7 @@ typedef int combo_t;
 typedef bool button_t;
 
 typedef struct {
-  char** combination;
+  char **combination;
   int count;
   int default_index;
 } option_combo_setting_t;
@@ -36,7 +37,7 @@ typedef struct {
     option_combo_setting_t combo;
     option_spin_setting_t spin;
     bool check_default;
-    const char* string_default;
+    const char *string_default;
   } data;
   union {
     combo_t *combo;
@@ -47,52 +48,63 @@ typedef struct {
   } value;
 } option_setting_t;
 
-option_setting_t option_setting_combo(
-  char* option_name,
-  char** combinations,
+void option_setting_combo(
+  option_setting_t *target,
+  char *option_name,
+  char **combinations,
   int combinations_count,
   int default_index,
   combo_t *combo
 );
-option_setting_t option_setting_spin(
-  const char* option_name,
+void option_setting_spin(
+  option_setting_t *target,
+  const char *option_name,
   int min,
   int max,
   int default_value,
   spin_t *spin
 );
-option_setting_t option_setting_check(
-  const char* option_name,
+void option_setting_check(
+  option_setting_t *target,
+  const char *option_name,
   bool default_value,
   check_t *check
 );
-option_setting_t option_setting_string(
-  const char* option_name,
-  const char* default_value,
+void option_setting_string(
+  option_setting_t *target,
+  const char *option_name,
+  const char *default_value,
   str_t *string
 );
-option_setting_t option_setting_button(
-  const char* option_name,
+void option_setting_button(
+  option_setting_t *target,
+  const char *option_name,
   button_t *button
 );
+void print_setting(option_setting_t setting);
+
+enum {
+  UCI_SIGNAL_NONE,
+  UCI_SIGNAL_QUIT,
+  UCI_SIGNAL_GO,
+  UCI_SIGNAL_GO_LOOP,
+  UCI_SIGNAL_STOP, // game stop, don't stop the program
+  
+  UCI_SIGNAL_COUNT
+};
 
 typedef struct {
+  size_t signal;
+  size_t current_state;
+
   char name[32];
   char author[32];
   bool debug;
 
-  atomic_int go;
-  atomic_int go_ready_receive;
-  atomic_int quit;
-  atomic_int stop;
-  atomic_int cleanup;
-
-  void* go_args; // TODO: fix the information flow so i don't need to do this trick
+  go_args_t go_args;
   position_t position;
-  struct uci_move *moves;
-  int moves_count;
-
-  option_setting_t option_settings[11];
+  ipc_moves_t *moves;
+option_setting_t option_settings[11];
 
   // options
   spin_t threads;
@@ -112,11 +124,10 @@ typedef struct {
   str_t uci_engineabout;
   // str_t uci_shredderbasespath;
   str_t uci_setpositionvalue;
-} uci_state;
+} uci_state_t;
 
-struct uci_move {
-  int from;
-  int to;
-};
+void uci_state_init(uci_state_t *state);
+void uci_state_deinit(uci_state_t *state);
+void apply_option(uci_state_t *state, char *name, char *buffer);
 
 #endif // UCI_STATE_H