Merge pull request #7 from ascent12/logging

Logging improvements
This commit is contained in:
Drew DeVault 2017-06-04 23:16:28 -04:00 committed by GitHub
commit 6298351d83
4 changed files with 32 additions and 61 deletions

View File

@ -39,6 +39,7 @@ endif()
add_definitions(-DWLR_GIT_VERSION=\"${GIT_COMMIT_HASH}\") add_definitions(-DWLR_GIT_VERSION=\"${GIT_COMMIT_HASH}\")
add_definitions(-DWLR_GIT_BRANCH=\"${GIT_BRANCH}\") add_definitions(-DWLR_GIT_BRANCH=\"${GIT_BRANCH}\")
add_definitions(-DWLR_SRC_DIR=\"${CMAKE_SOURCE_DIR}\")
string(TIMESTAMP CURRENT_DATE "%Y-%m-%d" UTC) string(TIMESTAMP CURRENT_DATE "%Y-%m-%d" UTC)
add_definitions(-DWLR_VERSION_DATE=\"${CURRENT_DATE}\") add_definitions(-DWLR_VERSION_DATE=\"${CURRENT_DATE}\")

View File

@ -10,7 +10,7 @@
#include "common/log.h" #include "common/log.h"
static bool colored = true; static bool colored = true;
static log_callback_t log_callback; static log_callback_t log_callback = wlr_log_stderr;
static const char *verbosity_colors[] = { static const char *verbosity_colors[] = {
[L_SILENT] = "", [L_SILENT] = "",
@ -21,46 +21,26 @@ static const char *verbosity_colors[] = {
void wlr_log_init(log_callback_t callback) { void wlr_log_init(log_callback_t callback) {
log_callback = callback; log_callback = callback;
// TODO: Use log callback
} }
void _wlr_vlog(const char *filename, int line, log_importance_t verbosity, void wlr_log_stderr(log_importance_t verbosity, const char *fmt, va_list args) {
const char *format, va_list args) {
// prefix the time to the log message // prefix the time to the log message
static struct tm result; struct tm result;
static time_t t; time_t t = time(NULL);
static struct tm *tm_info; struct tm *tm_info = localtime_r(&t, &result);
char buffer[26]; char buffer[26];
// get current time
t = time(NULL);
// convert time to local time (determined by the locale)
tm_info = localtime_r(&t, &result);
// generate time prefix // generate time prefix
strftime(buffer, sizeof(buffer), "%x %X - ", tm_info); strftime(buffer, sizeof(buffer), "%F %T - ", tm_info);
fprintf(stderr, "%s", buffer); fprintf(stderr, "%s", buffer);
unsigned int c = verbosity; unsigned c = (verbosity < L_LAST) ? verbosity : L_LAST - 1;
if (c > sizeof(verbosity_colors) / sizeof(char *) - 1) {
c = sizeof(verbosity_colors) / sizeof(char *) - 1;
}
if (colored && isatty(STDERR_FILENO)) { if (colored && isatty(STDERR_FILENO)) {
fprintf(stderr, "%s", verbosity_colors[c]); fprintf(stderr, "%s", verbosity_colors[c]);
} }
if (filename && line) { vfprintf(stderr, fmt, args);
const char *file = filename + strlen(filename);
while (file != filename && *file != '/') {
--file;
}
if (*file == '/') {
++file;
}
fprintf(stderr, "[%s:%d] ", file, line);
}
vfprintf(stderr, format, args);
if (colored && isatty(STDERR_FILENO)) { if (colored && isatty(STDERR_FILENO)) {
fprintf(stderr, "\x1B[0m"); fprintf(stderr, "\x1B[0m");
@ -68,33 +48,13 @@ void _wlr_vlog(const char *filename, int line, log_importance_t verbosity,
fprintf(stderr, "\n"); fprintf(stderr, "\n");
} }
void _wlr_log(const char *filename, int line, log_importance_t verbosity, const char* format, ...) { void _wlr_vlog(log_importance_t verbosity, const char *fmt, va_list args) {
va_list args; log_callback(verbosity, fmt, args);
va_start(args, format);
_wlr_vlog(filename, line, verbosity, format, args);
va_end(args);
} }
void wlr_log_errno(log_importance_t verbosity, char* format, ...) { void _wlr_log(log_importance_t verbosity, const char *fmt, ...) {
unsigned int c = verbosity;
if (c > sizeof(verbosity_colors) / sizeof(char *) - 1) {
c = sizeof(verbosity_colors) / sizeof(char *) - 1;
}
if (isatty(STDERR_FILENO)) {
fprintf(stderr, "%s", verbosity_colors[c]);
}
va_list args; va_list args;
va_start(args, format); va_start(args, fmt);
vfprintf(stderr, format, args); log_callback(verbosity, fmt, args);
va_end(args); va_end(args);
fprintf(stderr, ": ");
fprintf(stderr, "%s", strerror(errno));
if (isatty(STDERR_FILENO)) {
fprintf(stderr, "\x1B[0m");
}
fprintf(stderr, "\n");
} }

View File

@ -1,18 +1,26 @@
#ifndef _WLR_INTERNAL_COMMON_LOG_H #ifndef _WLR_INTERNAL_COMMON_LOG_H
#define _WLR_INTERNAL_COMMON_LOG_H #define _WLR_INTERNAL_COMMON_LOG_H
#include <stdbool.h> #include <stdbool.h>
#include <string.h>
#include <errno.h>
#include <wlr/common/log.h> #include <wlr/common/log.h>
void wlr_log_errno(log_importance_t verbosity, char* format, ...) __attribute__((format(printf,2,3))); #ifdef __GNUC__
#define ATTRIB_PRINTF(start, end) __attribute__((format(printf, start, end)))
#else
#define ATTRIB_PRINTF(start, end)
#endif
void wlr_log_errno(log_importance_t verbosity, char* format, ...) __attribute__((format(printf,2,3))); void _wlr_log(log_importance_t verbosity, const char *format, ...) ATTRIB_PRINTF(2, 3);
void _wlr_vlog(log_importance_t verbosity, const char *format, va_list args) ATTRIB_PRINTF(2, 0);
void _wlr_log(const char *filename, int line, log_importance_t verbosity, const char* format, ...) __attribute__((format(printf,4,5))); #define wlr_log(verb, fmt, ...) \
_wlr_log(verb, "[%s:%d] " fmt, __FILE__ + strlen(WLR_SRC_DIR) + 1, __LINE__, ##__VA_ARGS__)
#define wlr_log(VERBOSITY, FMT, ...) \ #define wlr_vlog(verb, fmt, args) \
_wlr_log(__FILE__, __LINE__, VERBOSITY, FMT, ##__VA_ARGS__) _wlr_vlog(verb, "[%s:%d] " fmt, __FILE__ + strlen(WLR_SRC_DIR) + 1, __LINE__, args)
#define wlr_vlog(VERBOSITY, FMT, VA_ARGS) \ #define wlr_log_errno(verb, fmt, ...) \
_wlr_vlog(__FILE__, __LINE__, VERBOSITY, FMT, VA_ARGS) wlr_log(verb, fmt ": %s", ##__VA_ARGS__, strerror(errno))
#endif #endif

View File

@ -8,10 +8,12 @@ typedef enum {
L_ERROR = 1, L_ERROR = 1,
L_INFO = 2, L_INFO = 2,
L_DEBUG = 3, L_DEBUG = 3,
L_LAST,
} log_importance_t; } log_importance_t;
typedef void (*log_callback_t)(log_importance_t importance, const char *fmt, va_list args); typedef void (*log_callback_t)(log_importance_t importance, const char *fmt, va_list args);
void init_log(log_callback_t callback); void wlr_init_log(log_callback_t callback);
void wlr_log_stderr(log_importance_t verbosity, const char *fmt, va_list args);
#endif #endif