imrpove logger code

src/Components/Modules/Logger.cpp

@@ -28,23 +28,28 @@ namespace Components
 
 		va_list va;
 		va_start(va, message);
-		vsnprintf_s(buf, _TRUNCATE, message, va);
+		const auto len = vsnprintf_s(buf, _TRUNCATE, message, va);
 		va_end(va);
 
+		if (len < 0)
+		{
+			return;
+		}
+
 		MessagePrint(channel, std::string{ buf });
 	}
 
 	void Logger::MessagePrint(const int channel, const std::string& msg)
 	{
-		static const auto shouldPrint = []() -> bool
+		static const auto print = []() -> bool
 		{
 			return Flags::HasFlag("stdout") || Loader::IsPerformingUnitTests();
 		}();
 
-		if (shouldPrint)
+		if (print)
 		{
-			std::printf("%s", msg.data());
 			std::fflush(stdout);
+			std::fputs(msg.data(), stdout);
 			return;
 		}
 

src/Components/Modules/Logger.hpp

@@ -83,27 +83,34 @@
 		struct FormatWithLocation
 		{
 			std::string_view format;
+#ifdef LOGGER_TRACE
 			std::source_location location;
-
-			FormatWithLocation(const std::string_view& fmt, std::source_location loc = std::source_location::current())
-				: format(fmt)
-				, location(std::move(loc))
+#endif
+#ifdef LOGGER_TRACE
+			FormatWithLocation(const std::string_view& fmt, std::source_location loc = std::source_location::current()) : format(fmt), location(std::move(loc))
+#else
+			FormatWithLocation(const std::string_view& fmt) : format(fmt)
+#endif
 			{
 			}
-
-			FormatWithLocation(const char* fmt, std::source_location loc = std::source_location::current())
-				: format(fmt)
-				, location(std::move(loc))
+#ifdef LOGGER_TRACE
+			FormatWithLocation(const char* fmt, std::source_location loc = std::source_location::current()) : format(fmt), location(std::move(loc))
+#else
+			FormatWithLocation(const char* fmt) : format(fmt)
+#endif
 			{
 			}
 		};
-
 		template <typename... Args>
 		static void Debug([[maybe_unused]] const FormatWithLocation& f, [[maybe_unused]] const Args&... args)
 		{
 #ifdef _DEBUG
 			(Utils::String::SanitizeFormatArgs(args), ...);
+#ifdef LOGGER_TRACE
 			DebugInternal(f.format, std::make_format_args(args...), f.location);
+#else
+			DebugInternal(f.format, std::make_format_args(args...));
+#endif
 #endif
 		}