Minimal "include" cleanup in utility files

Author: JavierJF

include/gen_utils.h

@@ -3,8 +3,11 @@
 
 #include <memory>
 #include <queue>
-#include "proxysql.h"
-#include "sqlite3db.h"
+
+#include "proxysql_structs.h"
+#include "proxysql_mem.h"
+
+#include "mysql.h"
 
 #define MIN_ARRAY_LEN 8
 #define MIN_ARRAY_DELETE_RATIO  8

include/proxysql_coredump.h

@@ -1,6 +1,8 @@
 #ifndef __PROXYSQL_COREDUMP_H
 #define __PROXYSQL_COREDUMP_H
+
 #include <unordered_set>
+#include <string>
 
 #define S1(x) #x
 #define S2(x) S1(x)

include/proxysql_debug.h

@@ -1,10 +1,6 @@
 #ifndef __PROXYSQL_DEBUG_H
 #define __PROXYSQL_DEBUG_H
 
-#include <chrono>
-#include <iostream>
-#include <atomic>
-
 #include "proxysql_macros.h"
 
 #if ENABLE_TIMER // this is defined in proxysql_macros.h

lib/gen_utils.cpp

@@ -1,8 +1,11 @@
 #include <vector>
 #include <memory>
 #include <sstream>
-#include "gen_utils.h"
 
+#include "sys/stat.h"
+
+#include "gen_utils.h"
+#include "sqlite3db.h"
 
 using std::vector;
 using std::unique_ptr;

lib/proxysql_coredump.cpp

@@ -1,8 +1,10 @@
 #include <string>
 #include <unordered_set>
+#include <unistd.h>
 
-#include "proxysql_coredump.h"
+#include "proxysql.h"
 #include "gen_utils.h"
+
 #if (defined(__i386__) || defined(__x86_64__) || defined(__ARM_ARCH_3__) || defined(__mips__)) && defined(__linux)
 // currently only support x86-32, x86-64, ARM, and MIPS on Linux
 #include "coredumper/coredumper.h"