Add namespaces

Author: Hind-M

include/serialize.hpp

@@ -1,13 +1,17 @@
 #pragma once
 
 #include <vector>
+
 #include "sparrow.hpp"
 
 #include "config/config.hpp"
 
-//TODO split serialize/deserialize fcts in two different files or just rename the current one?
-template <typename T>
-SPARROW_IPC_API std::vector<uint8_t> serialize_primitive_array(const sparrow::primitive_array<T>& arr);
+namespace sparrow_ipc
+{
+    //TODO split serialize/deserialize fcts in two different files or just rename the current one?
+    template <typename T>
+    SPARROW_IPC_API std::vector<uint8_t> serialize_primitive_array(const sparrow::primitive_array<T>& arr);
 
-template <typename T>
-SPARROW_IPC_API sparrow::primitive_array<T> deserialize_primitive_array(const std::vector<uint8_t>& buffer);
+    template <typename T>
+    SPARROW_IPC_API sparrow::primitive_array<T> deserialize_primitive_array(const std::vector<uint8_t>& buffer);
+}

include/utils.hpp

@@ -5,19 +5,22 @@
 #include <string_view>
 #include <utility>
 
-// TODO what to do with namespace?
-// TODO add tests for these?
-// TODO add namespace ? sparrow-ipc / detail or utils?
-#include "config/config.hpp"
-
-// Aligns a value to the next multiple of 8, as required by the Arrow IPC format for message bodies.
-SPARROW_IPC_API int64_t align_to_8(int64_t n);
+#include "Schema_generated.h"
 
-// Parse the format string
-SPARROW_IPC_API std::optional<int32_t> parse_format(std::string_view format_str, std::string_view sep);
+#include "config/config.hpp"
 
-SPARROW_IPC_API std::pair<org::apache::arrow::flatbuf::Type, flatbuffers::Offset<void>>
-get_flatbuffer_decimal_type(flatbuffers::FlatBufferBuilder& builder, std::string_view format_str, int32_t bitWidth);
+// TODO add tests
+// TODO in that case define #ifdef TESTING for the fcts here to be testable and available in anonymous namespaces
+namespace sparrow_ipc
+{
+    namespace utils
+    {
+        // Aligns a value to the next multiple of 8, as required by the Arrow IPC format for message bodies
+        SPARROW_IPC_API int64_t align_to_8(int64_t n);
 
-SPARROW_IPC_API std::pair<org::apache::arrow::flatbuf::Type, flatbuffers::Offset<void>>
-get_flatbuffer_type(flatbuffers::FlatBufferBuilder& builder, std::string_view format_str);
+        // Creates a Flatbuffers type from a format string
+        // This function maps a sparrow data type to the corresponding Flatbuffers type
+        SPARROW_IPC_API std::pair<org::apache::arrow::flatbuf::Type, flatbuffers::Offset<void>>
+        get_flatbuffer_type(flatbuffers::FlatBufferBuilder& builder, std::string_view format_str);
+    }
+}