Ditch that weird internal.h header

src/Makefile.am

@@ -31,9 +31,9 @@ apfl_internal_headers += context.h
 apfl_internal_headers += format.h
 apfl_internal_headers += gc.h
 apfl_internal_headers += hashmap.h
-apfl_internal_headers += internal.h
 apfl_internal_headers += resizable.h
 apfl_internal_headers += scope.h
+apfl_internal_headers += strings.h
 apfl_internal_headers += value.h
 
 EXTRA_DIST = $(apfl_internal_headers) apfl.h

src/context.c

@@ -7,8 +7,8 @@
 #include "context.h"
 #include "gc.h"
 #include "hashmap.h"
-#include "internal.h"
 #include "resizable.h"
+#include "strings.h"
 #include "value.h"
 
 static struct stack *

src/eval.c

@@ -7,7 +7,7 @@
 #include "compile.h"
 #include "context.h"
 #include "hashmap.h"
-#include "internal.h"
+#include "strings.h"
 #include "value.h"
 
 #define TRY(ex)                         \

src/expr.c

@@ -6,7 +6,6 @@
 
 #include "alloc.h"
 #include "format.h"
-#include "internal.h"
 
 #define TRY(x) do { if (!(x)) return false; } while (0)
 

src/parser.c

@@ -6,7 +6,6 @@
 
 #include "alloc.h"
 #include "resizable.h"
-#include "internal.h"
 
 struct apfl_parser {
     struct apfl_allocator allocator;

src/strings.c

@@ -7,8 +7,8 @@
 
 #include "alloc.h"
 #include "gc.h"
-#include "internal.h"
 #include "resizable.h"
+#include "strings.h"
 
 struct apfl_string_view
 apfl_string_view_from_view(struct apfl_string_view view)

src/strings.h

@@ -1,17 +1,13 @@
-#ifndef APFL_INTERNAL_H
-#define APFL_INTERNAL_H
+#ifndef APFL_STRINGS_H
+#define APFL_STRINGS_H
 
 #ifdef __cplusplus
 extern "C" {
 #endif
 
-#include <stdio.h>
-#include <stdlib.h>
-
+#include "apfl.h"
 #include "gc.h"
 
-// Internal use only functions
-
 struct apfl_string *apfl_string_move_into_new_gc_string(struct gc *gc, struct apfl_string *in);
 
 #ifdef __cplusplus

src/value.c

@@ -5,7 +5,6 @@
 #include "alloc.h"
 #include "context.h"
 #include "format.h"
-#include "internal.h"
 #include "resizable.h"
 #include "hashmap.h"
 #include "value.h"