Fix for #pragma asm

danmar · danmar · commit 1f4b84379d1e · 2017-05-17T21:58:46.000+02:00
diff --git a/lib/cppcheck.cpp b/lib/cppcheck.cpp
@@ -212,6 +212,7 @@ unsigned int CppCheck::processFile(const std::string& filename, const std::strin
 
         // Get directives
         preprocessor.setDirectives(tokens1);
+        preprocessor.simplifyPragmaAsm(&tokens1);
 
         preprocessor.setPlatformInfo(&tokens1);
 
diff --git a/lib/preprocessor.cpp b/lib/preprocessor.cpp
@@ -566,10 +566,6 @@ void Preprocessor::loadFiles(const simplecpp::TokenList &rawtokens, std::vector<
     simplecpp::OutputList outputList;
 
     tokenlists = simplecpp::load(rawtokens, files, dui, &outputList);
-
-    for (std::map<std::string, simplecpp::TokenList *>::iterator it = tokenlists.begin(); it != tokenlists.end(); ++it) {
-        Preprocessor::simplifyPragmaAsm(it->second);
-    }
 }
 
 void Preprocessor::removeComments()
@@ -886,6 +882,14 @@ unsigned int Preprocessor::calculateChecksum(const simplecpp::TokenList &tokens1
 }
 
 void Preprocessor::simplifyPragmaAsm(simplecpp::TokenList *tokenList)
+{
+    Preprocessor::simplifyPragmaAsmPrivate(tokenList);
+    for (std::map<std::string, simplecpp::TokenList *>::iterator it = tokenlists.begin(); it != tokenlists.end(); ++it) {
+        Preprocessor::simplifyPragmaAsmPrivate(it->second);
+    }
+}
+
+void Preprocessor::simplifyPragmaAsmPrivate(simplecpp::TokenList *tokenList)
 {
     // assembler code..
     for (simplecpp::Token *tok = tokenList->front(); tok; tok = tok->next) {
diff --git a/lib/preprocessor.h b/lib/preprocessor.h
@@ -171,10 +171,12 @@ class CPPCHECKLIB Preprocessor {
      */
     unsigned int calculateChecksum(const simplecpp::TokenList &tokens1, const std::string &toolinfo) const;
 
-    static void simplifyPragmaAsm(simplecpp::TokenList *tokenList);
+    void simplifyPragmaAsm(simplecpp::TokenList *tokenList);
 
 private:
 
+    static void simplifyPragmaAsmPrivate(simplecpp::TokenList *tokenList);
+
     /**
      * Remove space that has new line character on left or right side of it.
      *
diff --git a/test/testpreprocessor.cpp b/test/testpreprocessor.cpp
@@ -253,7 +253,7 @@ class TestPreprocessor : public TestFixture {
         std::vector<std::string> files;
         simplecpp::TokenList tokens(istr, files, filename, &outputList);
         tokens.removeComments();
-        Preprocessor::simplifyPragmaAsm(&tokens);
+        preprocessor0.simplifyPragmaAsm(&tokens);
         const std::set<std::string> configs(preprocessor0.getConfigs(tokens));
         preprocessor0.setDirectives(tokens);
         for (std::set<std::string>::const_iterator it = configs.begin(); it != configs.end(); ++it) {
