Add arena test. Refactor trivial data parser.

packages/logging/source/common/Log.cpp

@@ -105,6 +105,10 @@ namespace logging {
 		}
 	};
 	Logger::~Logger() {
+		if (mCondition && (mLevel == LogLevel::LogExpection || mLevel == LogLevel::LogAssertion)) {
+			return;
+		}
+
 		mStream.flush();
 		std::string msg = mStream.str();
 		mStream.str("");
@@ -116,24 +120,25 @@ namespace logging {
 		case LogLevel::LogWarning:
 		case LogLevel::LogError:
 		case LogLevel::LogTest:
+		case LogLevel::LogTitle:
 			break;
 		case LogLevel::LogExpection:
 			if (!mCondition) {
 				// Breaking runtime condition, show popup?
 			}
-			else {
-				msg.clear(); // Error message should not be displayed if condition was right
-			}
+			// already handled in early return
+			//else {
+			//	msg.clear(); // Error message should not be displayed if condition was right
+			//}
 			break;
 		case LogLevel::LogAssertion:
 			if (!mCondition) {
 				DEBUG_BREAK;
 			}
-			else {
-				msg.clear(); // Error message should not be displayed if condition was right
-			}
-			break;
-		case LogLevel::LogTitle:
+			// already handled in early return
+			//else {
+			//	msg.clear(); // Error message should not be displayed if condition was right
+			//}
 			break;
 		}
 

packages/memory/include/memory/Arena.h

@@ -25,12 +25,13 @@ namespace l::memory {
 	class Arena {
 	public:
 		Arena();
+		Arena(void* ptr, uint64_t size);
 		~Arena();
 
 		std::vector<MemoryBlock> mBlocks;
 	};
 
-	std::unique_ptr<Arena> CreateArena();
+	std::unique_ptr<Arena> CreateArena(void* ptr = nullptr, uint64_t size = 0);
 
 	// push some bytes onto the 'stack' - the way to allocate
 	void* ArenaPush(Arena* arena, uint64_t size);

packages/memory/source/common/Arena.cpp

@@ -19,13 +19,16 @@ namespace l::memory {
 		mBlocks.reserve(16);
 
 	}
+	Arena::Arena(void* ptr, uint64_t size) {
+		mBlocks.push_back({ ptr, size });
+	}
 
 	Arena::~Arena() {
 
 	}
 
-	std::unique_ptr<Arena> CreateArena() {
-		return std::make_unique<Arena>();
+	std::unique_ptr<Arena> CreateArena(void* ptr, uint64_t size) {
+		return std::make_unique<Arena>(ptr, size);
 	}
 
 

packages/memory/tests/common/ArenaTest.cpp

@@ -0,0 +1,18 @@
+#include "testing/Test.h"
+#include "logging/Log.h"
+
+#include "memory/Arena.h"
+
+using namespace l;
+
+TEST(Arena, Basic) {
+
+	auto arena = memory::CreateArena();
+
+	auto a = memory::ArenaPush(arena.get(), 10);
+
+	TEST_TRUE(a != nullptr, "");
+
+	return 0;
+}
+

packages/serialization/include/serialization/CommaSeparatedData.h → packages/serialization/include/serialization/TrivialData.h

@@ -6,7 +6,7 @@
 
 #include "logging/String.h"
 
-namespace l::storage {
+namespace l::serialization {
 
 	template<size_t size>
 	std::pair<size_t, std::vector<std::string>> ParseTrivialData(std::stringstream& data, std::string_view separators) {