Added .cl, .hxx, .tpp and .txx to list of supported file extensions - now matches the ones supported by Cppcheck internally / separated extension list by C/C++/header for future usage to match Cppcheck as well

Author: firewave

README.md

@@ -62,6 +62,7 @@ Deployment.
 - Only scan files which actually exist. (Contribution by @firewave)
 - Use unique file names for temporary files used for analysis. (Contribution by @firewave)
 - 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)
 
 ### 1.5.1 - 2020-11-12
 

resources/META-INF/plugin.xml

@@ -66,6 +66,7 @@
       - Only scan files which actually exist. (Contribution by @firewave)
       - Use unique file names for temporary files used for analysis. (Contribution by @firewave)
       - 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
     ]]>
   </change-notes>
 

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

@@ -21,6 +21,9 @@
 import javax.xml.parsers.ParserConfigurationException;
 import java.io.File;
 import java.io.IOException;
+import java.util.ArrayList;
+import java.util.Arrays;
+import java.util.Collections;
 import java.util.List;
 
 public class CppcheckInspection extends LocalInspectionTool {
@@ -92,21 +95,51 @@ private static String prependIncludeDir(@NotNull String cppcheckOptions, @NotNul
         return String.format("-I\"%s\" %s", path, cppcheckOptions);
     }
 
+    // TODO: get the list of supported extensions from Cppcheck if it provides that information
+    // TODO: extend list by extensions configured within CLion
+    private final static List<String> supportedCExtensions = new ArrayList<>(Arrays.asList(
+            "c",
+            "cl"));
+
+    private final static List<String> supportedCPPExtensions = new ArrayList<>(Arrays.asList(
+            "cc",
+            "cp",
+            "cpp",
+            "c++",
+            "cxx",
+            "hh",
+            "hpp",
+            "hxx",
+            "tpp",
+            "txx"));
+
+    private final static List<String> supportedHeaderExtensions = new ArrayList<>(Collections.singletonList(
+            "h"));
+
     private static boolean isCFamilyFile(@NotNull final VirtualFile file) {
+        return isCFile(file) || isCPPFile(file) || isHeaderFile(file);
+    }
+
+    static private boolean isFile(@NotNull final VirtualFile file, @NotNull final List<String> supportedExtensions)
+    {
         final String fileExtension = file.getExtension();
         if (fileExtension == null) {
             return false;
         }
 
         final String lowerFileExtension = fileExtension.toLowerCase();
-        return lowerFileExtension.equals("c") ||
-                lowerFileExtension.equals("cc") ||
-                lowerFileExtension.equals("cp") ||
-                lowerFileExtension.equals("cpp") ||
-                lowerFileExtension.equals("c++") ||
-                lowerFileExtension.equals("cxx") ||
-                lowerFileExtension.equals("h") ||
-                lowerFileExtension.equals("hh") ||
-                lowerFileExtension.equals("hpp");
+        return supportedExtensions.contains(lowerFileExtension);
+    }
+
+    static private boolean isCFile(@NotNull final VirtualFile file) {
+        return isFile(file, supportedCExtensions);
+    }
+
+    static private boolean isCPPFile(@NotNull final VirtualFile file) {
+        return isFile(file, supportedCPPExtensions);
+    }
+
+    static private boolean isHeaderFile(@NotNull final VirtualFile file) {
+        return isFile(file, supportedHeaderExtensions);
     }
 }