fix: changed utilities back to how it was before

include/rconpp/rcon.h

@@ -1,7 +1,7 @@
 #pragma once
 
 #ifdef _WIN32
-#pragma warning( disable : 4251 ); // 4251 warns when we export classes or structures with stl member variables
+#pragma warning( disable : 4251 ) // 4251 warns when we export classes or structures with stl member variables
 #endif
 
 #include "export.h"

src/rconpp/utilities.cpp

@@ -15,7 +15,7 @@ rconpp::packet rconpp::form_packet(const std::string_view data, int32_t id, int3
 
 	std::vector<char> temp_data(data_size + 4); /* Create a vector that exactly the size of the packet length. */
 
-	std::memcpy(temp_data.data(), &data_size, sizeof(data_size)); /* Copy size into it */
+	std::memcpy(temp_data.data() + 0, &data_size, sizeof(data_size)); /* Copy size into it */
 	std::memcpy(temp_data.data() + sizeof(data_size), &id, sizeof(id)); /* Copy id into it */
 	std::memcpy(temp_data.data() + sizeof(data_size) + sizeof(id), &type, sizeof(type)); /* Copy type into it */
 	std::memcpy(temp_data.data() + sizeof(data_size) + sizeof(id) + sizeof(type), data.data(), data.size()); /* Copy data into it */

unittest/test.cpp

@@ -10,7 +10,7 @@ int main() {
 		rconpp::rcon_client client(std::getenv("RCON_TESTING_IP"), std::stoi(std::getenv("RCON_TESTING_PORT")),
 					   std::getenv("RCON_TESTING_PASSWORD"));
 
-		client.on_log = [](const std::string_view& log) {
+		client.on_log = [](const std::string_view log) {
 			std::cout << log << "\n";
 		};