From: Stephen Seo <seo.disparate@gmail.com>
Date: Tue, 22 Oct 2019 10:27:55 +0000 (+0900)
Subject: Refactor logging
X-Git-Tag: 1.0~142
X-Git-Url: https://git.seodisparate.com/stephenseo/search/main.js?a=commitdiff_plain;h=0f435613bc0d3c960712f2ff8b5bc5c18399a3c9;p=UDPConnection

Refactor logging

Removed redundant checking of logging type in log_impl* .
---

diff --git a/cpp_impl/src/UDPC_Defines.hpp b/cpp_impl/src/UDPC_Defines.hpp
index 25bf295..62bd37f 100644
--- a/cpp_impl/src/UDPC_Defines.hpp
+++ b/cpp_impl/src/UDPC_Defines.hpp
@@ -140,60 +140,18 @@ private:
 
     template<typename... Targs>
     void log_impl(UDPC_LoggingType type, Targs... args) { // {{{
-        switch(loggingType.load()) {
-        case UDPC_LoggingType::UDPC_SILENT:
-            return;
+        switch(type) {
         case UDPC_LoggingType::UDPC_ERROR:
-            if(type == UDPC_LoggingType::UDPC_ERROR) {
-                std::cerr << "ERROR: ";
-            } else {
-                return;
-            }
+            std::cerr << "ERROR: ";
             break;
         case UDPC_LoggingType::UDPC_WARNING:
-            switch(type) {
-            case UDPC_LoggingType::UDPC_ERROR:
-                std::cerr << "ERROR: ";
-                break;
-            case UDPC_LoggingType::UDPC_WARNING:
-                std::cerr << "WARNING: ";
-                break;
-            default:
-                return;
-            }
+            std::cerr << "WARN: ";
             break;
         case UDPC_LoggingType::UDPC_VERBOSE:
-            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;
-            default:
-                return;
-            }
+            std::cerr << "VERB: ";
             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;
-            case UDPC_LoggingType::UDPC_INFO:
-                std::cerr << "INFO: ";
-                break;
-            default:
-                return;
-            }
+            std::cerr << "INFO: ";
             break;
         default:
             return;
@@ -208,33 +166,7 @@ private:
 
     template<typename T, typename... Targs>
     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;
-            }
-            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;
-        }
+        std::cerr << value;
         log_impl_next(type, args...);
     } // }}}