Code cleaning

Author: MorganCaron

modules/Math/Random.mpp

@@ -7,8 +7,10 @@ export namespace CppUtils::Math
 	template<std::integral Integer>
 	inline auto getRandomNumberInInterval(Integer min, Integer max) -> Integer
 	{
-		static auto randomDevice = std::random_device{};
-		static auto engine = std::mt19937{randomDevice()};
+		thread_local auto engine = [] {
+			static auto randomDevice = std::random_device{};
+			return std::mt19937{randomDevice() ^ std::hash<std::thread::id>{}(std::this_thread::get_id())};
+		}();
 		auto distribution = std::uniform_int_distribution<Integer>{min, max};
 		auto randomNumber = distribution(engine);
 		return randomNumber;

modules/Math/Utility.mpp

@@ -43,11 +43,6 @@ export namespace CppUtils::Math
 	static_assert(not isEqual(0.1 + 0.2, 0.29));
 	static_assert(not isEqual(0.1 + 0.2, 0.31));
 
-	[[nodiscard]] inline constexpr auto roundToNDecimals(std::floating_point auto number, std::size_t nbDecimals) -> decltype(number)
-	{
-		return std::round(number * static_cast<decltype(number)>(nbDecimals)) / static_cast<decltype(number)>(nbDecimals);
-	}
-
 	[[nodiscard]] inline constexpr auto isBetween(auto value, auto low, auto high) noexcept -> bool
 	{
 		return low <= value and value <= high;

modules/Network/Socket.mpp

@@ -393,8 +393,8 @@ export namespace CppUtils::Network
 		template<>
 		[[nodiscard]] inline auto receive<std::vector<std::byte>>(std::size_t bufferSize) -> std::optional<std::vector<std::byte>>
 		{
-			auto buffer = std::vector<std::byte>{bufferSize};
-			buffer.reserve(bufferSize);
+			auto buffer = std::vector<std::byte>{};
+			buffer.resize(bufferSize);
 			for (auto totalBytesReceived = 0uz; totalBytesReceived < bufferSize;)
 			{
 #if defined(OS_WINDOWS)

tests/Thread/Scheduler.mpp

@@ -189,7 +189,7 @@ export namespace CppUtils::UnitTest::Thread::Scheduler
 
 		suite.addTest("Stress test with many tasks", [&] {
 			auto scheduler = CppUtils::Thread::Scheduler{};
-			const auto numTasks = 100'000uz;
+			const auto numTasks = 10'000uz;
 			auto executedCount = std::atomic_size_t{0};
 
 			for (auto i = 0uz; i < numTasks; ++i)