From 78904d2bcc7345d74a59dd4dd44da685207cf787 Mon Sep 17 00:00:00 2001 From: Marco Lizza Date: Sun, 20 Jan 2019 22:23:07 +0100 Subject: Fixing `TraceLog` function to work on levels. --- src/utils.c | 54 ++++++++++++++++++++++++++---------------------------- 1 file changed, 26 insertions(+), 28 deletions(-) (limited to 'src/utils.c') diff --git a/src/utils.c b/src/utils.c index 1cf5c0ff..9a312296 100644 --- a/src/utils.c +++ b/src/utils.c @@ -92,53 +92,51 @@ void SetTraceLogCallback(TraceLogCallback callback) } // Show trace log messages (LOG_INFO, LOG_WARNING, LOG_ERROR, LOG_DEBUG) -void TraceLog(int msgType, const char *text, ...) +void TraceLog(TraceLogType logType, const char *text, ...) { #if defined(SUPPORT_TRACELOG) - char buffer[MAX_TRACELOG_BUFFER_SIZE] = { 0 }; + if (logType < logTypeLevel) { // Message has level below current threshold, don't emit. + return; + } + va_list args; va_start(args, text); if (logCallback) { - logCallback(msgType, text, args); + logCallback(logType, text, args); va_end(args); return; } - switch(msgType) - { - case LOG_INFO: strcpy(buffer, "INFO: "); break; - case LOG_ERROR: strcpy(buffer, "ERROR: "); break; - case LOG_WARNING: strcpy(buffer, "WARNING: "); break; - case LOG_DEBUG: strcpy(buffer, "DEBUG: "); break; - case LOG_OTHER: strcpy(buffer, "OTHER: "); break; - default: break; - } - - strcat(buffer, text); - strcat(buffer, "\n"); - #if defined(PLATFORM_ANDROID) - switch(msgType) + switch(logType) { - case LOG_INFO: if (logTypeFlags & LOG_INFO) __android_log_vprint(ANDROID_LOG_INFO, "raylib", buffer, args); break; - case LOG_WARNING: if (logTypeFlags & LOG_WARNING) __android_log_vprint(ANDROID_LOG_WARN, "raylib", buffer, args); break; - case LOG_ERROR: if (logTypeFlags & LOG_ERROR) __android_log_vprint(ANDROID_LOG_ERROR, "raylib", buffer, args); break; - case LOG_DEBUG: if (logTypeFlags & LOG_DEBUG) __android_log_vprint(ANDROID_LOG_DEBUG, "raylib", buffer, args); break; - case LOG_OTHER: if (logTypeFlags & LOG_OTHER) __android_log_vprint(ANDROID_LOG_VERBOSE, "raylib", buffer, args); break; + case LOG_TRACE: __android_log_vprint(ANDROID_LOG_VERBOSE, "raylib", text, args); break; + case LOG_DEBUG: __android_log_vprint(ANDROID_LOG_DEBUG, "raylib", text, args); break; + case LOG_INFO: __android_log_vprint(ANDROID_LOG_INFO, "raylib", text, args); ; break; + case LOG_WARNING: __android_log_vprint(ANDROID_LOG_WARN, "raylib", text, args); break; + case LOG_ERROR: __android_log_vprint(ANDROID_LOG_ERROR, "raylib", text, args); break; + case LOG_FATAL: __android_log_vprint(ANDROID_LOG_FATAL, "raylib", text, args); break; default: break; } #else - switch(msgType) + char buffer[MAX_TRACELOG_BUFFER_SIZE] = { 0 }; + + switch(logType) { - case LOG_INFO: if (logTypeFlags & LOG_INFO) vprintf(buffer, args); break; - case LOG_WARNING: if (logTypeFlags & LOG_WARNING) vprintf(buffer, args); break; - case LOG_ERROR: if (logTypeFlags & LOG_ERROR) vprintf(buffer, args); break; - case LOG_DEBUG: if (logTypeFlags & LOG_DEBUG) vprintf(buffer, args); break; - case LOG_OTHER: if (logTypeFlags & LOG_OTHER) vprintf(buffer, args); break; + case LOG_TRACE: strcpy(buffer, "TRACE: "); break; + case LOG_DEBUG: strcpy(buffer, "DEBUG: "); break; + case LOG_INFO: strcpy(buffer, "INFO: "); ; break; + case LOG_WARNING: strcpy(buffer, "WARN: "); break; + case LOG_ERROR: strcpy(buffer, "ERROR: "); break; + case LOG_FATAL: strcpy(buffer, "FATAL: "); break; default: break; } + + strcat(buffer, text); + strcat(buffer, "\n"); + vprintf(buffer, args); #endif va_end(args); -- cgit v1.2.3 From 932e9dbd090e48d96be8636beaaebcf11c2f9117 Mon Sep 17 00:00:00 2001 From: Marco Lizza Date: Sun, 20 Jan 2019 22:24:09 +0100 Subject: Fixing logging level configuration API. --- src/raylib.h | 6 +++--- src/utils.c | 10 +++++----- 2 files changed, 8 insertions(+), 8 deletions(-) (limited to 'src/utils.c') diff --git a/src/raylib.h b/src/raylib.h index 2210915f..325a1777 100644 --- a/src/raylib.h +++ b/src/raylib.h @@ -819,7 +819,7 @@ typedef enum { } NPatchType; // Callbacks to be implemented by users -typedef void (*TraceLogCallback)(int msgType, const char *text, va_list args); +typedef void (*TraceLogCallback)(TraceLogType logType, const char *text, va_list args); #if defined(__cplusplus) extern "C" { // Prevents name mangling of functions @@ -899,9 +899,9 @@ RLAPI Color Fade(Color color, float alpha); // Color fade- // Misc. functions RLAPI void SetConfigFlags(unsigned char flags); // Setup window configuration flags (view FLAGS) -RLAPI void SetTraceLog(unsigned char types); // Enable trace log message types (bit flags based) +RLAPI void SetTraceLogLevel(TraceLogType logType); // Set the current threshold (minimum) log level. RLAPI void SetTraceLogCallback(TraceLogCallback callback); // Set a trace log callback to enable custom logging bypassing raylib's one -RLAPI void TraceLog(int logType, const char *text, ...); // Show trace log messages (LOG_INFO, LOG_WARNING, LOG_ERROR, LOG_DEBUG) +RLAPI void TraceLog(TraceLogType logType, const char *text, ...); // Show trace log messages (LOG_INFO, LOG_WARNING, LOG_ERROR, LOG_DEBUG) RLAPI void TakeScreenshot(const char *fileName); // Takes a screenshot of current screen (saved a .png) RLAPI int GetRandomValue(int min, int max); // Returns a random value between min and max (both included) diff --git a/src/utils.c b/src/utils.c index 9a312296..e9f46187 100644 --- a/src/utils.c +++ b/src/utils.c @@ -52,8 +52,8 @@ // Global Variables Definition //---------------------------------------------------------------------------------- -// Log types messages supported flags (bit based) -static unsigned char logTypeFlags = LOG_INFO | LOG_WARNING | LOG_ERROR; +// Log types messages +static TraceLogType logTypeLevel = LOG_INFO; static TraceLogCallback logCallback = NULL; #if defined(PLATFORM_ANDROID) @@ -79,10 +79,10 @@ static int android_close(void *cookie); // Module Functions Definition - Utilities //---------------------------------------------------------------------------------- -// Enable trace log message types (bit flags based) -void SetTraceLog(unsigned char types) +// Set the current threshold (minimum) log level. +void SetTraceLogLevel(TraceLogType logType) { - logTypeFlags = types; + logTypeLevel = logType; } // Set a trace log callback to enable custom logging bypassing raylib's one -- cgit v1.2.3 From f1d13aa469230960130038871a9ce785bbe55c71 Mon Sep 17 00:00:00 2001 From: Marco Lizza Date: Sun, 20 Jan 2019 22:24:41 +0100 Subject: Adding function to control the "exit-on-error" behaviour. --- src/raylib.h | 1 + src/utils.c | 9 ++++++++- 2 files changed, 9 insertions(+), 1 deletion(-) (limited to 'src/utils.c') diff --git a/src/raylib.h b/src/raylib.h index 325a1777..9929a96d 100644 --- a/src/raylib.h +++ b/src/raylib.h @@ -900,6 +900,7 @@ RLAPI Color Fade(Color color, float alpha); // Color fade- // Misc. functions RLAPI void SetConfigFlags(unsigned char flags); // Setup window configuration flags (view FLAGS) RLAPI void SetTraceLogLevel(TraceLogType logType); // Set the current threshold (minimum) log level. +RLAPI void SetTraceLogExit(TraceLogType logType); // Set the exit threshold (minimum) log level. RLAPI void SetTraceLogCallback(TraceLogCallback callback); // Set a trace log callback to enable custom logging bypassing raylib's one RLAPI void TraceLog(TraceLogType logType, const char *text, ...); // Show trace log messages (LOG_INFO, LOG_WARNING, LOG_ERROR, LOG_DEBUG) RLAPI void TakeScreenshot(const char *fileName); // Takes a screenshot of current screen (saved a .png) diff --git a/src/utils.c b/src/utils.c index e9f46187..ab327272 100644 --- a/src/utils.c +++ b/src/utils.c @@ -54,6 +54,7 @@ // Log types messages static TraceLogType logTypeLevel = LOG_INFO; +static TraceLogType logTypeExit = LOG_ERROR; static TraceLogCallback logCallback = NULL; #if defined(PLATFORM_ANDROID) @@ -85,6 +86,12 @@ void SetTraceLogLevel(TraceLogType logType) logTypeLevel = logType; } +// Set the exit threshold (minimum) log level. +void SetTraceLogExit(TraceLogType logType) +{ + logTypeExit = logType; +} + // Set a trace log callback to enable custom logging bypassing raylib's one void SetTraceLogCallback(TraceLogCallback callback) { @@ -141,7 +148,7 @@ void TraceLog(TraceLogType logType, const char *text, ...) va_end(args); - if (msgType == LOG_ERROR) exit(1); // If LOG_ERROR message, exit program + if (logType >= logTypeExit) exit(1); // If exit message, exit program #endif // SUPPORT_TRACELOG } -- cgit v1.2.3 From 343fef4aa4532e21bd74ed84e0339756eb5ab897 Mon Sep 17 00:00:00 2001 From: Marco Lizza Date: Mon, 21 Jan 2019 10:05:40 +0100 Subject: Changing enums, now referenced as int. --- src/raylib.h | 8 ++++---- src/utils.c | 10 +++++----- 2 files changed, 9 insertions(+), 9 deletions(-) (limited to 'src/utils.c') diff --git a/src/raylib.h b/src/raylib.h index 9929a96d..739199ba 100644 --- a/src/raylib.h +++ b/src/raylib.h @@ -819,7 +819,7 @@ typedef enum { } NPatchType; // Callbacks to be implemented by users -typedef void (*TraceLogCallback)(TraceLogType logType, const char *text, va_list args); +typedef void (*TraceLogCallback)(int logType, const char *text, va_list args); #if defined(__cplusplus) extern "C" { // Prevents name mangling of functions @@ -899,10 +899,10 @@ RLAPI Color Fade(Color color, float alpha); // Color fade- // Misc. functions RLAPI void SetConfigFlags(unsigned char flags); // Setup window configuration flags (view FLAGS) -RLAPI void SetTraceLogLevel(TraceLogType logType); // Set the current threshold (minimum) log level. -RLAPI void SetTraceLogExit(TraceLogType logType); // Set the exit threshold (minimum) log level. +RLAPI void SetTraceLogLevel(int logType); // Set the current threshold (minimum) log level. +RLAPI void SetTraceLogExit(int logType); // Set the exit threshold (minimum) log level. RLAPI void SetTraceLogCallback(TraceLogCallback callback); // Set a trace log callback to enable custom logging bypassing raylib's one -RLAPI void TraceLog(TraceLogType logType, const char *text, ...); // Show trace log messages (LOG_INFO, LOG_WARNING, LOG_ERROR, LOG_DEBUG) +RLAPI void TraceLog(int logType, const char *text, ...); // Show trace log messages (LOG_INFO, LOG_WARNING, LOG_ERROR, LOG_DEBUG) RLAPI void TakeScreenshot(const char *fileName); // Takes a screenshot of current screen (saved a .png) RLAPI int GetRandomValue(int min, int max); // Returns a random value between min and max (both included) diff --git a/src/utils.c b/src/utils.c index ab327272..ffce11fb 100644 --- a/src/utils.c +++ b/src/utils.c @@ -53,8 +53,8 @@ //---------------------------------------------------------------------------------- // Log types messages -static TraceLogType logTypeLevel = LOG_INFO; -static TraceLogType logTypeExit = LOG_ERROR; +static int logTypeLevel = LOG_INFO; +static int logTypeExit = LOG_ERROR; static TraceLogCallback logCallback = NULL; #if defined(PLATFORM_ANDROID) @@ -81,13 +81,13 @@ static int android_close(void *cookie); //---------------------------------------------------------------------------------- // Set the current threshold (minimum) log level. -void SetTraceLogLevel(TraceLogType logType) +void SetTraceLogLevel(int logType) { logTypeLevel = logType; } // Set the exit threshold (minimum) log level. -void SetTraceLogExit(TraceLogType logType) +void SetTraceLogExit(int logType) { logTypeExit = logType; } @@ -99,7 +99,7 @@ void SetTraceLogCallback(TraceLogCallback callback) } // Show trace log messages (LOG_INFO, LOG_WARNING, LOG_ERROR, LOG_DEBUG) -void TraceLog(TraceLogType logType, const char *text, ...) +void TraceLog(int logType, const char *text, ...) { #if defined(SUPPORT_TRACELOG) if (logType < logTypeLevel) { // Message has level below current threshold, don't emit. -- cgit v1.2.3