chore: some reorganising

README.md

@@ -36,6 +36,9 @@ All file names, function names... (even internals) should follow snake case
 convention and have a prefix unique to that module, starting with `vieter_`.
 For example, the `cron` modules uses the `vieter_cron_` prefix for everything.
 
+Header files should only import what they explicitely need. If some function is
+only used in a .c file, the import should be placed in the .c file instead.
+
 ### Testing
 
 This library uses [Acutest](https://github.com/mity/acutest) for its tests.

include/vieter_cron.h

@@ -1,7 +1,6 @@
 #ifndef VIETER_CRON
 #define VIETER_CRON
 
-#include <stdbool.h>
 #include <stdint.h>
 #include <stdlib.h>
 #include <string.h>

include/vieter_heap.h

@@ -1,7 +1,6 @@
 #ifndef VIETER_HEAP
 #define VIETER_HEAP
 
-#include <stddef.h>
 #include <stdint.h>
 
 typedef struct vieter_heap vieter_heap;

src/cron/vieter_cron_parse.c

@@ -1,4 +1,8 @@
 #include "vieter_cron.h"
+#include <stdbool.h>
+#include <stdlib.h>
+#include <string.h>
+#include <time.h>
 
 // This prefix is needed to properly compile
 const uint8_t parse_month_days[] = {31, 28, 31, 30, 31, 30,