stephenseo/UDPConnection
1
0
Fork 0
Code Issues Pull requests Projects Releases 2 Packages 1 Wiki Activity Actions

Refactor logging

Browse source 
Removed redundant checking of logging type in log_impl* .
This commit is contained in:
Stephen Seo 2019-10-22 19:27:55 +09:00
parent b11d87ca12
commit 0f435613bc
1 changed files with 6 additions and 74 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

72
cpp_impl/src/UDPC_Defines.hpp
View file

@ -140,53 +140,15 @@ private:
template<typename... Targs> template<typename... Targs>
void log_impl(UDPC_LoggingType type, Targs... args) { // {{{ void log_impl(UDPC_LoggingType type, Targs... args) { // {{{
switch(loggingType.load()) {
case UDPC_LoggingType::UDPC_SILENT:
return;
case UDPC_LoggingType::UDPC_ERROR:
if(type == UDPC_LoggingType::UDPC_ERROR) {
std::cerr << "ERROR: ";
} else {
return;
}
break;
case UDPC_LoggingType::UDPC_WARNING:
switch(type) { switch(type) {
case UDPC_LoggingType::UDPC_ERROR: case UDPC_LoggingType::UDPC_ERROR:
std::cerr << "ERROR: "; std::cerr << "ERROR: ";
break; break;
case UDPC_LoggingType::UDPC_WARNING: case UDPC_LoggingType::UDPC_WARNING:
std::cerr << "WARNING: "; std::cerr << "WARN: ";
break;
default:
return;
}
break; break;
case UDPC_LoggingType::UDPC_VERBOSE: case UDPC_LoggingType::UDPC_VERBOSE:
switch(type) { std::cerr << "VERB: ";
case UDPC_LoggingType::UDPC_ERROR:
std::cerr << "ERROR: ";
break;
case UDPC_LoggingType::UDPC_WARNING:
std::cerr << "WARNING: ";
break;
case UDPC_LoggingType::UDPC_VERBOSE:
std::cerr << "VERBOSE: ";
break;
default:
return;
}
break;
case UDPC_LoggingType::UDPC_INFO:
switch(type) {
case UDPC_LoggingType::UDPC_ERROR:
std::cerr << "ERROR: ";
break;
case UDPC_LoggingType::UDPC_WARNING:
std::cerr << "WARNING: ";
break;
case UDPC_LoggingType::UDPC_VERBOSE:
std::cerr << "VERBOSE: ";
break; break;
case UDPC_LoggingType::UDPC_INFO: case UDPC_LoggingType::UDPC_INFO:
std::cerr << "INFO: "; std::cerr << "INFO: ";
@ -194,10 +156,6 @@ private:
default: default:
return; return;
} }
break;
default:
return;
}
log_impl_next(type, args...); log_impl_next(type, args...);
} // }}} } // }}}
@ -208,33 +166,7 @@ private:
template<typename T, typename... Targs> template<typename T, typename... Targs>
void log_impl_next(UDPC_LoggingType type, T value, Targs... args) { // {{{ void log_impl_next(UDPC_LoggingType type, T value, Targs... args) { // {{{
switch(loggingType.load()) {
case UDPC_LoggingType::UDPC_SILENT:
return;
case UDPC_LoggingType::UDPC_ERROR:
if(type == UDPC_LoggingType::UDPC_ERROR) {
std::cerr << value; std::cerr << value;
}
break;
case UDPC_LoggingType::UDPC_WARNING:
if(type == UDPC_LoggingType::UDPC_ERROR || type == UDPC_LoggingType::UDPC_WARNING) {
std::cerr << value;
}
break;
case UDPC_LoggingType::UDPC_VERBOSE:
if(type == UDPC_LoggingType::UDPC_ERROR || type == UDPC_LoggingType::UDPC_WARNING
|| type == UDPC_LoggingType::UDPC_VERBOSE) {
std::cerr << value;
}
break;
case UDPC_LoggingType::UDPC_INFO:
if(type == UDPC_LoggingType::UDPC_ERROR || type == UDPC_LoggingType::UDPC_WARNING
|| type == UDPC_LoggingType::UDPC_VERBOSE
|| type == UDPC_LoggingType::UDPC_INFO) {
std::cerr << value;
}
break;
}
log_impl_next(type, args...); log_impl_next(type, args...);
} // }}} } // }}}