fix severity colorization

esp. information as red was disturbing

Author: rurban

cli/cmdlineparser.cpp

@@ -1394,7 +1394,7 @@ CmdLineParser::Result CmdLineParser::parseFromArgs(int argc, const char* const a
 
     // Default template format..
     if (mSettings.templateFormat.empty()) {
-        mSettings.templateFormat = "{bold}{file}:{line}:{column}: {red}{inconclusive:{magenta}}{severity}:{inconclusive: inconclusive:}{default} {message} [{id}]{reset}\\n{code}";
+        mSettings.templateFormat = "{bold}{file}:{line}:{column}: {severity}:{inconclusive: inconclusive:}{default} {message} [{id}]{reset}\\n{code}";
         if (mSettings.templateLocation.empty())
             mSettings.templateLocation = "{bold}{file}:{line}:{column}: {dim}note:{reset} {info}\\n{code}";
     }

lib/errorlogger.cpp

@@ -171,8 +171,8 @@ ErrorMessage::ErrorMessage(const tinyxml2::XMLElement * const errmsg)
     id = attr ? attr : unknown;
 
     attr = errmsg->Attribute("severity");
-    severity = attr ? severityFromString(attr) : Severity::none;
-
+    severity = attr ? severityFromString(attr) : Severity::none;        
+      
     attr = errmsg->Attribute("cwe");
     // cppcheck-suppress templateInstantiation - TODO: fix this - see #11631
     cwe.id = attr ? strToInt<unsigned short>(attr) : 0;
@@ -637,7 +637,9 @@ std::string ErrorMessage::toString(bool verbose, const std::string &templateForm
         findAndReplace(result, replaceFrom, replaceWith);
         pos1 = result.find("{inconclusive:", pos1);
     }
-    findAndReplace(result, "{severity}", severityToString(severity));
+    std::string sseverity = coloredSeverityToString(severity);
+    replaceColors(sseverity);
+    findAndReplace(result, "{severity}", sseverity);
     findAndReplace(result, "{cwe}", std::to_string(cwe.id));
     findAndReplace(result, "{message}", verbose ? mVerboseMessage : mShortMessage);
     findAndReplace(result, "{remark}", remark);

lib/errortypes.cpp

@@ -72,6 +72,31 @@ std::string severityToString(Severity severity)
     throw InternalError(nullptr, "Unknown severity");
 }
 
+std::string coloredSeverityToString(Severity severity)
+{
+    switch (severity) {
+    case Severity::none:
+        return "";
+    case Severity::error:
+        return "{red}error";
+    case Severity::warning:
+        return "{magenta}warning";
+    case Severity::style:
+        return "{bold}style";
+    case Severity::performance:
+        return "{bold}performance";
+    case Severity::portability:
+        return "{bold}portability";
+    case Severity::information:
+        return "{green}information";
+    case Severity::debug:
+        return "debug";
+    case Severity::internal:
+        return "internal";
+    }
+    throw InternalError(nullptr, "Unknown severity");
+}
+
 // TODO: bail out on invalid severity
 Severity severityFromString(const std::string& severity)
 {

lib/errortypes.h

@@ -120,6 +120,7 @@ enum class Severity : std::uint8_t {
 };
 
 CPPCHECKLIB std::string severityToString(Severity severity);
+CPPCHECKLIB std::string coloredSeverityToString(Severity severity);
 CPPCHECKLIB Severity severityFromString(const std::string &severity);
 
 struct CWE {