CppCheck: no need to preprocess markup files as they are not actual source code (#5896)

It should not be necessary to preprocess markup files. We only need to
tokenize them so function calls can be marked as used.

Author: firewave

lib/cppcheck.cpp

@@ -632,6 +632,18 @@ unsigned int CppCheck::checkFile(const std::string& filename, const std::string
     CheckUnusedFunctions checkUnusedFunctions(nullptr, nullptr, nullptr);
 
     try {
+        if (mSettings.library.markupFile(filename)) {
+            Tokenizer tokenizer(mSettings, this);
+            if (fileStream)
+                tokenizer.list.createTokens(*fileStream, filename);
+            else {
+                std::ifstream in(filename);
+                tokenizer.list.createTokens(in, filename);
+            }
+            checkUnusedFunctions.getFileInfo(&tokenizer, &mSettings);
+            return EXIT_SUCCESS;
+        }
+
         Preprocessor preprocessor(mSettings, this);
         std::set<std::string> configurations;
 
@@ -662,13 +674,6 @@ unsigned int CppCheck::checkFile(const std::string& filename, const std::string
             return mExitCode;
         }
 
-        if (mSettings.library.markupFile(filename)) {
-            Tokenizer tokenizer(mSettings, this, &preprocessor);
-            tokenizer.createTokens(std::move(tokens1));
-            checkUnusedFunctions.getFileInfo(&tokenizer, &mSettings);
-            return EXIT_SUCCESS;
-        }
-
         if (!preprocessor.loadFiles(tokens1, files))
             return mExitCode;