From 75d85714e692d8ab95e05880959cbae6cc540360 Mon Sep 17 00:00:00 2001 From: Ioan Sucan Date: Fri, 12 Apr 2013 09:38:47 +0300 Subject: [PATCH] rename log levels to prevent conflicts with syslog.h (#2) --- include/console_bridge/console.h | 18 +++++++++--------- src/console.cpp | 6 +++--- 2 files changed, 12 insertions(+), 12 deletions(-) diff --git a/include/console_bridge/console.h b/include/console_bridge/console.h index 6bf4833..43209e4 100644 --- a/include/console_bridge/console.h +++ b/include/console_bridge/console.h @@ -61,13 +61,13 @@ \} */ -#define logError(fmt, ...) console_bridge::log(__FILE__, __LINE__, console_bridge::LOG_ERROR, fmt, ##__VA_ARGS__) +#define logError(fmt, ...) console_bridge::log(__FILE__, __LINE__, console_bridge::CONSOLE_BRIDGE_LOG_ERROR, fmt, ##__VA_ARGS__) -#define logWarn(fmt, ...) console_bridge::log(__FILE__, __LINE__, console_bridge::LOG_WARN, fmt, ##__VA_ARGS__) +#define logWarn(fmt, ...) console_bridge::log(__FILE__, __LINE__, console_bridge::CONSOLE_BRIDGE_LOG_WARN, fmt, ##__VA_ARGS__) -#define logInform(fmt, ...) console_bridge::log(__FILE__, __LINE__, console_bridge::LOG_INFO, fmt, ##__VA_ARGS__) +#define logInform(fmt, ...) console_bridge::log(__FILE__, __LINE__, console_bridge::CONSOLE_BRIDGE_LOG_INFO, fmt, ##__VA_ARGS__) -#define logDebug(fmt, ...) console_bridge::log(__FILE__, __LINE__, console_bridge::LOG_DEBUG, fmt, ##__VA_ARGS__) +#define logDebug(fmt, ...) console_bridge::log(__FILE__, __LINE__, console_bridge::CONSOLE_BRIDGE_LOG_DEBUG, fmt, ##__VA_ARGS__) /** \brief Message namespace. This contains classes needed to @@ -78,11 +78,11 @@ namespace console_bridge /** \brief The set of priorities for message logging */ enum LogLevel { - LOG_DEBUG = 0, - LOG_INFO, - LOG_WARN, - LOG_ERROR, - LOG_NONE + CONSOLE_BRIDGE_LOG_DEBUG = 0, + CONSOLE_BRIDGE_LOG_INFO, + CONSOLE_BRIDGE_LOG_WARN, + CONSOLE_BRIDGE_LOG_ERROR, + CONSOLE_BRIDGE_LOG_NONE }; /** \brief Generic class to handle output from a piece of diff --git a/src/console.cpp b/src/console.cpp index 86ba91b..95920a5 100644 --- a/src/console.cpp +++ b/src/console.cpp @@ -48,7 +48,7 @@ struct DefaultOutputHandler { output_handler_ = static_cast(&std_output_handler_); previous_output_handler_ = output_handler_; - logLevel_ = console_bridge::LOG_WARN; + logLevel_ = console_bridge::CONSOLE_BRIDGE_LOG_WARN; } console_bridge::OutputHandlerSTD std_output_handler_; @@ -133,7 +133,7 @@ static const char* LogLevelString[4] = {"Debug: ", "Info: ", "Warning: ", " void console_bridge::OutputHandlerSTD::log(const std::string &text, LogLevel level, const char *filename, int line) { - if (level >= LOG_WARN) + if (level >= CONSOLE_BRIDGE_LOG_WARN) { std::cerr << LogLevelString[level] << text << std::endl; std::cerr << " at line " << line << " in " << filename << std::endl; @@ -165,7 +165,7 @@ void console_bridge::OutputHandlerFile::log(const std::string &text, LogLevel le if (file_) { fprintf(file_, "%s%s\n", LogLevelString[level], text.c_str()); - if(level >= LOG_WARN) + if(level >= CONSOLE_BRIDGE_LOG_WARN) fprintf(file_, " at line %d in %s\n", line, filename); fflush(file_); }