common: refactor sway_log()

This removes most preprocessor logic, leaving it only it the header.
This commit is contained in:
Eric Engestrom 2016-05-02 15:10:22 +01:00
parent 8a6b64e471
commit cc9d1cacbb
2 changed files with 9 additions and 12 deletions

View file

@ -61,11 +61,7 @@ void sway_abort(const char *format, ...) {
sway_terminate(EXIT_FAILURE);
}
#ifndef NDEBUG
void _sway_log(const char *filename, int line, log_importance_t verbosity, const char* format, ...) {
#else
void _sway_log(log_importance_t verbosity, const char* format, ...) {
#endif
if (verbosity <= v) {
unsigned int c = verbosity;
if (c > sizeof(verbosity_colors) / sizeof(char *) - 1) {
@ -76,13 +72,14 @@ void _sway_log(log_importance_t verbosity, const char* format, ...) {
fprintf(stderr, "%s", verbosity_colors[c]);
}
if (filename && line) {
char *file = strdup(filename);
fprintf(stderr, "[%s:%d] ", basename(file), line);
free(file);
}
va_list args;
va_start(args, format);
#ifndef NDEBUG
char *file = strdup(filename);
fprintf(stderr, "[%s:%d] ", basename(file), line);
free(file);
#endif
vfprintf(stderr, format, args);
va_end(args);