diff --git a/common/command.cc b/common/command.cc index 206a4d30..d332375a 100644 --- a/common/command.cc +++ b/common/command.cc @@ -131,9 +131,9 @@ void CommandHandler::setupContext(Context *ctx) } if (vm.count("quiet")) { - log_streams.push_back(std::make_pair(&std::cerr, LogLevel::WARNING)); + log_streams.push_back(std::make_pair(&std::cerr, LogLevel::WARNING_MSG)); } else { - log_streams.push_back(std::make_pair(&std::cerr, LogLevel::LOG)); + log_streams.push_back(std::make_pair(&std::cerr, LogLevel::LOG_MSG)); } if (vm.count("log")) { @@ -141,7 +141,7 @@ void CommandHandler::setupContext(Context *ctx) logfile = std::ofstream(logfilename); if (!logfile) log_error("Failed to open log file '%s' for writing.\n", logfilename.c_str()); - log_streams.push_back(std::make_pair(&logfile, LogLevel::LOG)); + log_streams.push_back(std::make_pair(&logfile, LogLevel::LOG_MSG)); } if (vm.count("force")) { @@ -285,8 +285,8 @@ void CommandHandler::conflicting_options(const boost::program_options::variables void CommandHandler::printFooter() { - int warning_count = get_or_default(message_count_by_level, LogLevel::WARNING, 0), - error_count = get_or_default(message_count_by_level, LogLevel::ERROR, 0); + int warning_count = get_or_default(message_count_by_level, LogLevel::WARNING_MSG, 0), + error_count = get_or_default(message_count_by_level, LogLevel::ERROR_MSG, 0); if (warning_count > 0 || error_count > 0) log_always("%d warning%s, %d error%s\n", warning_count, warning_count == 1 ? "" : "s", error_count, error_count == 1 ? "" : "s"); diff --git a/common/log.cc b/common/log.cc index 0a75b020..01aec79a 100644 --- a/common/log.cc +++ b/common/log.cc @@ -84,7 +84,7 @@ std::string vstringf(const char *fmt, va_list ap) return string; } -void logv(const char *format, va_list ap, LogLevel level = LogLevel::LOG) +void logv(const char *format, va_list ap, LogLevel level = LogLevel::LOG_MSG) { // // Trim newlines from the beginning @@ -132,7 +132,7 @@ void log_always(const char *format, ...) { va_list ap; va_start(ap, format); - logv(format, ap, LogLevel::ALWAYS); + logv(format, ap, LogLevel::ALWAYS_MSG); va_end(ap); } @@ -140,7 +140,7 @@ void log(const char *format, ...) { va_list ap; va_start(ap, format); - logv(format, ap, LogLevel::LOG); + logv(format, ap, LogLevel::LOG_MSG); va_end(ap); } @@ -148,7 +148,7 @@ void log_info(const char *format, ...) { va_list ap; va_start(ap, format); - logv_prefixed("Info: ", format, ap, LogLevel::INFO); + logv_prefixed("Info: ", format, ap, LogLevel::INFO_MSG); va_end(ap); } @@ -156,7 +156,7 @@ void log_warning(const char *format, ...) { va_list ap; va_start(ap, format); - logv_prefixed("Warning: ", format, ap, LogLevel::WARNING); + logv_prefixed("Warning: ", format, ap, LogLevel::WARNING_MSG); va_end(ap); } @@ -164,7 +164,7 @@ void log_error(const char *format, ...) { va_list ap; va_start(ap, format); - logv_prefixed("ERROR: ", format, ap, LogLevel::ERROR); + logv_prefixed("ERROR: ", format, ap, LogLevel::ERROR_MSG); if (log_error_atexit) log_error_atexit(); @@ -184,7 +184,7 @@ void log_nonfatal_error(const char *format, ...) { va_list ap; va_start(ap, format); - logv_prefixed("ERROR: ", format, ap, LogLevel::ERROR); + logv_prefixed("ERROR: ", format, ap, LogLevel::ERROR_MSG); va_end(ap); had_nonfatal_error = true; } diff --git a/common/log.h b/common/log.h index 52158f18..5745df0d 100644 --- a/common/log.h +++ b/common/log.h @@ -44,11 +44,11 @@ struct log_execution_error_exception enum class LogLevel { - LOG, - INFO, - WARNING, - ERROR, - ALWAYS + LOG_MSG, + INFO_MSG, + WARNING_MSG, + ERROR_MSG, + ALWAYS_MSG }; extern std::vector> log_streams;