Renamed Logger.h to GFSDK_Logger.h since it's a shared header.

Removed the narrow char methods.
Removed the floating log functions.
Changed the ILogger::log() method to use varargs and printf() formatting.

1 files changed, 7 insertions, 67 deletions

diff --git a/src/InternalLogger.h b/src/InternalLogger.h
index d10f263..334519c 100644
--- a/src/InternalLogger.h
+++ b/src/InternalLogger.h
@@ -1,6 +1,6 @@
 #pragma once
 
-#include "Logger.h"
+#include "GFSDK_Logger.h"
 #include <cstdarg>
 #include <cstdio>
 #include <wtypes.h>
@@ -15,8 +15,8 @@ public:
 	InternalLogger(nv::LogSeverity loggingLevel);
 	virtual ~InternalLogger() = default;
 
-	virtual void log(const char* text, nv::LogSeverity severity, const char* filename, int linenumber) override;
-	virtual void log(const wchar_t* text, nv::LogSeverity severity, const wchar_t* filename, int linenumber) override;
+//	virtual void log(nv::LogSeverity severity, const char* filename, int linenumber, const char* text, ...) override;
+	virtual void log(nv::LogSeverity severity, const wchar_t* filename, int linenumber, const wchar_t* text, ...) override;
 
 	nv::LogSeverity getLoggingLevel();
 	void setLoggingLevel(nv::LogSeverity newLevel);
@@ -34,75 +34,15 @@ private:
 
 extern nv::ILogger* g_UserLogger;
 
-namespace WaveworksInternal
-{
-	inline void log(nv::LogSeverity severity, const wchar_t* filename, int linenumber, const wchar_t* format, ...)
-	{
-		wchar_t buffer[1024];
-
-		va_list args;
-		va_start(args, format);
-#if NV_GCC_FAMILY
-#pragma GCC diagnostic push
-#pragma GCC diagnostic ignored "-Wformat-nonliteral"
-#endif
-		_vsnwprintf_s(buffer, sizeof(buffer), format, args);
-#if NV_GCC_FAMILY
-#pragma GCC diagnostic pop
-#endif
-		va_end(args);
-
-
-		// Only output to one logger at a time. May not be the desired behaviour.
-		if (g_UserLogger)
-		{
-			g_UserLogger->log(buffer, severity, filename, linenumber);
-		}
-		else
-		{
-			InternalLogger::GetInstance()->log(buffer, severity, filename, linenumber);
-		}
-
-	}
-
-	inline void log(nv::LogSeverity severity, const char* filename, int linenumber, const char* format, ...)
-	{
-		char buffer[1024];
-
-		va_list args;
-		va_start(args, format);
-#if NV_GCC_FAMILY
-#pragma GCC diagnostic push
-#pragma GCC diagnostic ignored "-Wformat-nonliteral"
-#endif
-		vsnprintf_s(buffer, sizeof(buffer), format, args);
-#if NV_GCC_FAMILY
-#pragma GCC diagnostic pop
-#endif
-		va_end(args);
-
-		// Only output to one logger at a time. May not be the desired behaviour.
-		if (g_UserLogger)
-		{
-			g_UserLogger->log(buffer, severity, filename, linenumber);
-		}
-		else
-		{
-			InternalLogger::GetInstance()->log(buffer, severity, filename, linenumber);
-		}
-	}
-}
-
-
 
 #ifndef NV_LOG
 /// <param name="__VA_ARGS__">format, ...</param>
-#define NV_LOG(...) WaveworksInternal::log(nv::LogSeverity::kInfo, __DEF_FILE__, __LINE__, __VA_ARGS__)
+#define NV_LOG(...) if (g_UserLogger) { g_UserLogger->log(nv::LogSeverity::kInfo, __DEF_FILE__, __LINE__, __VA_ARGS__); } else { InternalLogger::GetInstance()->log(nv::LogSeverity::kInfo, __DEF_FILE__, __LINE__, __VA_ARGS__); }
 /// <param name="__VA_ARGS__">format, ...</param>
-#define NV_WARN(...) WaveworksInternal::log(nv::LogSeverity::kWarning, __DEF_FILE__, __LINE__, __VA_ARGS__)
+#define NV_WARN(...) if (g_UserLogger) { g_UserLogger->log(nv::LogSeverity::kWarning, __DEF_FILE__, __LINE__, __VA_ARGS__); } else { InternalLogger::GetInstance()->log(nv::LogSeverity::kWarning, __DEF_FILE__, __LINE__, __VA_ARGS__); }
 /// <param name="__VA_ARGS__">format, ...</param>
-#define NV_ERROR(...) WaveworksInternal::log(nv::LogSeverity::kError, __DEF_FILE__, __LINE__, __VA_ARGS__)
+#define NV_ERROR(...) if (g_UserLogger) { g_UserLogger->log(nv::LogSeverity::kError, __DEF_FILE__, __LINE__, __VA_ARGS__); } else { InternalLogger::GetInstance()->log(nv::LogSeverity::kError, __DEF_FILE__, __LINE__, __VA_ARGS__); }
 /// <param name="__VA_ARGS__">format, ...</param>
-#define NV_FATAL(...) WaveworksInternal::log(nv::LogSeverity::kFatal, __DEF_FILE__, __LINE__, __VA_ARGS__)
+#define NV_FATAL(...) if (g_UserLogger) { g_UserLogger->log(nv::LogSeverity::kFatal, __DEF_FILE__, __LINE__, __VA_ARGS__); } else { InternalLogger::GetInstance()->log(nv::LogSeverity::kFatal, __DEF_FILE__, __LINE__, __VA_ARGS__); }
 
 #endif
\ No newline at end of file