Clang doesn't like ntohl (I misread it as being 64 bit); swapped to generic template

Author: SimonCahill

include/libdbc/utils/utils.hpp

@@ -126,6 +126,30 @@ constexpr bool  isWhitespaceOrEmpty(const T& str) {
 	return std::all_of(str.begin(), str.end(), [](char c) { return std::isspace(c); });
 }
 
+/**
+ * @brief Swaps the endianness of a multi-byte value.
+ * 
+ * @tparam T The type of the value.
+ * @param value The value to swap.
+ * @return T The value with swapped endianness.
+ * 
+ * @copyright 2025 Procyon Systems Inh. Simon Cahill (s.cahill@procyon-systems.de)
+ */
+template<typename T>
+T                swapEndianness(T value) {
+	T swappedBytes;
+
+	char* valuePtr = reinterpret_cast<char*>(&value);
+	char* swappedPtr = reinterpret_cast<char*>(&swappedBytes);
+
+	auto sizeInBytes = sizeof(T);
+	for (size_t i = 0; i < sizeInBytes; i++) {
+		swappedPtr[sizeInBytes - 1 - i] = valuePtr[i];
+	}
+
+	return swappedBytes;
+}
+
 class String {
 public:
 	static std::string trim(const std::string& line) { return Utils::trim(line); }

src/message.cpp

@@ -1,5 +1,3 @@
-#include <arpa/inet.h>
-
 #include <cstddef>
 #include <cstdint>
 #include <libdbc/message.hpp>
@@ -8,6 +6,8 @@
 #include <string>
 #include <vector>
 
+#include <libdbc/utils/utils.hpp>
+
 namespace Libdbc {
 
 constexpr unsigned ONE_BYTE = 8;
@@ -39,7 +39,7 @@ Message::ParseSignalsStatus Message::parse_signals(const std::vector<uint8_t>& d
 	for (std::size_t i = 0; i < size; i++) {
 		data_big_endian = (data_big_endian << ONE_BYTE) | (uint64_t)data[i];
 	}
-	data_little_endian = ntohl(data_little_endian);
+	data_little_endian = Utils::swapEndianness(data_little_endian);
 
 	// TODO: does this also work on a big endian machine?