[fix] Fix includes according to iwyu

* All files that change in this commit are working after the changes
suggested from iwyu.
* For shellout.{c,h} and svsem.c changes to headers are not possible
since the headers suggested by iwyu are private and would break the
build of the library.

Author: innerout

.gitignore

@@ -1,2 +1,4 @@
 .ccls-cache/
 build
+.cache
+compile_commands.json

include/mba/cfg.h

@@ -25,6 +25,7 @@ extern "C" {
 #include <mba/allocator.h>
 #include <mba/linkedlist.h>
 #include <mba/text.h>
+struct allocator;
 
 struct cfg {
 	struct linkedlist list;

include/mba/csv.h

@@ -20,6 +20,7 @@ extern "C" {
 #endif /* WIN32 */
 #endif /* LIBMBA_API */
 
+#include <stddef.h>
 #include <stdio.h>
 #include <mba/text.h>
 

include/mba/daemon.h

@@ -3,7 +3,6 @@
 
 #include <stdio.h>
 #include <sys/types.h>
-#include <sys/stat.h>
 
 extern FILE *logfp;
 

include/mba/diff.h

@@ -22,6 +22,7 @@ extern "C" {
 
 #include <mba/varray.h>
 #include <mba/hashmap.h> /* cmp_fn */
+struct varray;
 
 typedef const void *(*idx_fn)(const void *s, int idx, void *context);
 

include/mba/hashmap.h

@@ -22,6 +22,7 @@ extern "C" {
 
 #include <mba/iterator.h>
 #include <mba/allocator.h>
+#include <stddef.h>
 
 #if USE_WCHAR
 #define hash_text hash_wcs
@@ -36,8 +37,6 @@ typedef int (*cmp_fn)(const void *object1, const void *object2, void *context);
 
 extern const int table_sizes[];
 
-struct entry;
-
 struct hashmap {
 	int table_size_index;
 	ref_t hash;

include/mba/misc.h

@@ -7,6 +7,7 @@ extern "C" {
 
 #ifndef _WIN32
 
+#include <stddef.h>
 #include <sys/types.h>
 
 int copen(const char *pathname, int flags, mode_t mode, int *cre);

include/mba/shellout.h

@@ -8,8 +8,9 @@
 extern "C" {
 #endif
 
-#include <sys/types.h>
 #include <termios.h>
+#include <sys/types.h>
+#include <stddef.h>
 
 #define SHO_FLAGS_INTERACT 0x0001
 #define SHO_FLAGS_ISATTY   0x0100

include/mba/svcond.h

@@ -10,7 +10,8 @@ extern "C" {
 #endif
 
 #include <mba/svsem.h>
-#include <mba/pool.h>
+struct allocator;
+struct pool;
 
 typedef struct {
 	struct pool *sempool;

include/mba/svsem.h

@@ -8,11 +8,8 @@
 extern "C" {
 #endif
 
-#include <sys/types.h>
-#include <sys/sem.h> /* for SEM_UNDO */
-#include <fcntl.h>
-#include <mba/pool.h>
 #include <mba/varray.h>
+struct pool;
 
 #define O_UNDO 0x80000
 

include/mba/text.h

@@ -170,8 +170,9 @@ LIBMBA_API int _fputws(const wchar_t *buf, FILE *stream);
 
 #else
 
-#include <string.h>
+#include <stddef.h>
 #include <ctype.h>
+struct allocator;
 
 #define TEOF EOF
 typedef int tint_t;

src/cfg.c

@@ -37,6 +37,8 @@
 #include <mba/cfg.h>
 #include <mba/text.h>
 
+#include <bits/types/mbstate_t.h>
+
 #if USE_WCHAR
 
 #define T1PMNF(m,s) PMNF((m), ": %ls", (s))

src/csv.c

@@ -22,16 +22,16 @@
  * OTHER DEALINGS IN THE SOFTWARE.
  */
 
-#include <stdlib.h>
 #include <string.h>
 #include <stdio.h>
+#include <stddef.h>
 #include <ctype.h>
 #include <errno.h>
-#include <wchar.h>
 #include <wctype.h>
 
 #include <mba/msgno.h>
 #include <mba/csv.h>
+#include <bits/types/wint_t.h>
 
 #define ST_START     1
 #define ST_COLLECT   2

src/hashmap.c

@@ -24,13 +24,12 @@
 #include <stddef.h>
 #include <string.h>
 #include <errno.h>
-#include <wchar.h>
 
 #include <mba/msgno.h>
 #include <mba/iterator.h>
 #include <mba/allocator.h>
 #include <mba/hashmap.h>
-
+#include <bits/types/wint_t.h>
 
 #define HAL(h) ((struct allocator *)((h)->al ? (char *)(h) - (ptrdiff_t)(h)->al : NULL))
 

src/text.c

@@ -22,6 +22,7 @@
  * OTHER DEALINGS IN THE SOFTWARE.
  */
 
+#include <bits/types/mbstate_t.h>
 #include <stdlib.h>
 #include <stdio.h>
 #include <stddef.h>