Fix violation of rule of 5 in JsonObject (#4)

* Add declarations for JsonObject's special functions

* Define copy/move constructors

* Define copy/move assignment operators

* Implicitly create bytes

* Implicitly create bytes in exchange()

* Fix includes lol (utility, cstring)

* Destroy() objects

Author: TymianekPL

cppjson/include/cppjson/object.hpp

@@ -25,6 +25,10 @@ namespace cppjson
 	{
 	  public:
 		explicit JsonObject();
+        JsonObject(const JsonObject& other);
+        JsonObject(JsonObject&& other);
+        JsonObject& operator=(const JsonObject& other);
+        JsonObject& operator=(JsonObject&& other);
 		~JsonObject();
 
 		template <typename T>

cppjson/src/object.cpp

@@ -1,11 +1,45 @@
 #include "cppjson/object.hpp"
 #include <new>
 #include <stdexcept>
+#include <cstring>
+#include <utility>
 
 constexpr std::size_t DataStorageSize = std::max({sizeof(std::string), sizeof(cppjson::Object), sizeof(double), sizeof(bool)});
 
 cppjson::JsonObject::JsonObject() : _dataStorage(static_cast<std::byte*>(::operator new(DataStorageSize))) {}
 
+cppjson::JsonObject::JsonObject(const cppjson::JsonObject& other)
+{
+	if (other._dataStorage == nullptr) return;
+	
+	this->_dataType = other._dataType;
+	this->_dataStorage = static_cast<std::byte*>(::operator new(DataStorageSize));
+	std::memcpy(this->_dataStorage, other._dataStorage, DataStorageSize);
+}
+cppjson::JsonObject::JsonObject(JsonObject&& other)
+{
+     this->_dataType = std::exchange(other._dataType, cppjson::JsonType::Null);
+     this->_dataStorage = std::exchange(other._dataStorage, static_cast<std::byte*>(::operator new(DataStorageSize)));
+}
+cppjson::JsonObject& cppjson::JsonObject::operator=(const cppjson::JsonObject& other)
+{
+    if (&other != this)
+    {
+		this->_dataType = other._dataType;
+		this->_dataStorage = static_cast<std::byte*>(::operator new(DataStorageSize));
+	    std::memcpy(this->_dataStorage, other._dataStorage, DataStorageSize);
+	}
+    return *this;
+}
+cppjson::JsonObject& cppjson::JsonObject::operator=(cppjson::JsonObject&& other)
+{
+    if (&other != this)
+    {
+		this->_dataType = std::exchange(other._dataType, cppjson::JsonType::Null);
+		this->_dataStorage = std::exchange(other._dataStorage, static_cast<std::byte*>(::operator new(DataStorageSize)));
+	}
+    return *this;
+}
 cppjson::JsonObject::~JsonObject()
 {
 	this->Destroy();
@@ -15,13 +49,16 @@ cppjson::JsonObject::~JsonObject()
 void cppjson::JsonObject::Destroy(void)
 {
 	using std::string;
+	using cppjson::Object;
 
 	switch (std::exchange(this->_dataType, JsonType::Null))
 	{
 	case JsonType::Null:
 	case JsonType::Number:
 	case JsonType::Bool: break;
-	case JsonType::String: DangerousAs<std::string>().~string();
+	case JsonType::String: DangerousAs<std::string>().~string(); break;
+	case JsonType::Object: DangerousAs<cppjson::Object>().~Object(); break;
+	// TODO: Array
 	}
 }