Move includes around

Author: Hind-M

include/serialize.hpp

@@ -1,10 +1,15 @@
 #pragma once
 
+#include <cstdint>
 #include <optional>
+#include <string>
 #include <vector>
 
 #include "sparrow.hpp"
 
+#include "Message_generated.h"
+#include "Schema_generated.h"
+
 #include "config/config.hpp"
 
 namespace sparrow_ipc

include/serialize_null_array.hpp

@@ -1,26 +1,10 @@
 #pragma once
 
-// TODO check needs of all these below
-#include <cstdint>
-#include <cstring>
-#include <optional>
-#include <stdexcept>
-#include <string>
-#include <vector>
-
-#include "sparrow.hpp"
-
-// TODO check needs of these two
-#include "Message_generated.h"
-#include "Schema_generated.h"
-
 #include "serialize.hpp"
-#include "utils.hpp"
 
 namespace sparrow_ipc
 {
     // TODO move to cpp if not templated
-    // TODO add comments and review
 
     // This function serializes a sparrow::null_array into a byte vector compliant
     // with the Apache Arrow IPC Streaming Format. It mirrors the structure of

include/serialize_primitive_array.hpp

@@ -1,16 +1,6 @@
 #pragma once
 
-#include <cstdint>
 #include <cstring>
-#include <optional>
-#include <stdexcept>
-#include <string>
-#include <vector>
-
-#include "sparrow.hpp"
-
-#include "Message_generated.h"
-#include "Schema_generated.h"
 
 #include "serialize.hpp"
 #include "utils.hpp"

src/serialize.cpp

@@ -1,11 +1,5 @@
-// TODO check need of all these includes
-#include <cstdint>
 #include <cstring>
 #include <stdexcept>
-#include <string>
-
-#include "Message_generated.h"
-#include "Schema_generated.h"
 
 #include "serialize.hpp"
 #include "utils.hpp"