Merge pull request #63 from firewave/info

fixed "information" messages not being shown at all

Author: firewave

README.md

@@ -69,6 +69,7 @@ Deployment.
 - Properly handle `debug` messages generated by `--debug-warnings`. (Contribution by @firewave)
 - Added `.cl`, `.hxx`, `.tpp` and `.txx` to list of supported file extensions - now matches the ones supported by Cppcheck internally. (Contribution by @firewave)
 - Internally suppress `unusedFunction` and `unusedStructMember`  (for header files only) warnings to avoid false positives. (Contribution by @firewave)
+- Fixed `information` messages not being shown at all. (Contribution by @firewave)
 
 ### 1.5.1 - 2020-11-12
 

resources/META-INF/plugin.xml

@@ -67,6 +67,7 @@
       - Properly handle "debug" messages generated by --debug-warnings. (Contribution by @firewave)
       - Added .cl, .hxx, .tpp and .txx to list of supported file extensions - now matches the ones supported by Cppcheck internally. (Contribution by @firewave)
       - Internally suppress "unusedFunction" and "unusedStructMember" (for header files only) warnings to avoid false positives. (Contribution by @firewave)
+      - Fixed "information" messages not being shown at all. (Contribution by @firewave)
     ]]>
   </change-notes>
 

src/com/github/johnthagen/cppcheck/CppCheckInspectionImpl.java

@@ -44,9 +44,9 @@ private static ProblemHighlightType severityToHighlightType(@NotNull final Strin
             case "performance":
             case "portability":
             case "debug":
-                return ProblemHighlightType.WEAK_WARNING;
+            // INFORMATION problems are not shown in the IDE at all so we need to treat them as weak warnings
             case "information":
-                return ProblemHighlightType.INFORMATION;
+                return ProblemHighlightType.WEAK_WARNING;
 
             // If the severity is not understood (changes in Cppcheck), return ERROR.
             default: