fixed #14275 - map simplecpp errors to individual IDs [skip ci]

firewave · firewave · commit ea70b6a1a1b3 · 2025-11-17T21:09:35.000+01:00
diff --git a/lib/cppcheck.cpp b/lib/cppcheck.cpp
@@ -1213,7 +1213,7 @@ unsigned int CppCheck::checkInternal(const FileWithDetails& file, const std::str
 
                 if (!hasValidConfig && currCfg == *configurations.rbegin()) {
                     // If there is no valid configuration then report error..
-                    preprocessor.error(o.location.file(), o.location.line, o.msg);
+                    preprocessor.error(o.location.file(), o.location.line, o.msg, o.type);
                 }
                 continue;
 
diff --git a/lib/errorlogger.cpp b/lib/errorlogger.cpp
@@ -45,6 +45,7 @@
 const std::set<std::string> ErrorLogger::mCriticalErrorIds{
     "cppcheckError",
     "cppcheckLimit",
+    "includeNestedTooDeeply",
     "internalAstError",
     "instantiationError",
     "internalError",
@@ -53,6 +54,7 @@ const std::set<std::string> ErrorLogger::mCriticalErrorIds{
     "premium-invalidLicense",
     "preprocessorErrorDirective",
     "syntaxError",
+    "unhandledChar",
     "unknownMacro"
 };
 
diff --git a/lib/preprocessor.cpp b/lib/preprocessor.cpp
@@ -312,7 +312,7 @@ void Preprocessor::inlineSuppressions(SuppressionList &suppressions)
         ::addInlineSuppressions(filedata->tokens, mSettings, suppressions, err);
     }
     for (const BadInlineSuppression &bad : err) {
-        error(bad.file, bad.line, bad.errmsg);
+        error(bad.file, bad.line, bad.errmsg, simplecpp::Output::ERROR); // TODO: use individual (non-fatal) ID
     }
 }
 
@@ -860,7 +860,7 @@ bool Preprocessor::reportOutput(const simplecpp::OutputList &outputList, bool sh
         case simplecpp::Output::ERROR:
             hasError = true;
             if (!startsWith(out.msg,"#error") || showerror)
-                error(out.location.file(), out.location.line, out.msg);
+                error(out.location.file(), out.location.line, out.msg, out.type);
             break;
         case simplecpp::Output::WARNING:
         case simplecpp::Output::PORTABILITY_BACKSLASH:
@@ -877,21 +877,47 @@ bool Preprocessor::reportOutput(const simplecpp::OutputList &outputList, bool sh
         case simplecpp::Output::SYNTAX_ERROR:
         case simplecpp::Output::UNHANDLED_CHAR_ERROR:
             hasError = true;
-            error(out.location.file(), out.location.line, out.msg);
+            error(out.location.file(), out.location.line, out.msg, out.type);
             break;
         case simplecpp::Output::EXPLICIT_INCLUDE_NOT_FOUND:
         case simplecpp::Output::FILE_NOT_FOUND:
         case simplecpp::Output::DUI_ERROR:
             hasError = true;
-            error("", 0, out.msg);
+            error("", 0, out.msg, out.type);
             break;
         }
     }
 
     return hasError;
 }
 
-void Preprocessor::error(const std::string &filename, unsigned int linenr, const std::string &msg)
+static std::string simplecppErrToId(simplecpp::Output::Type type)
+{
+    switch (type) {
+        case simplecpp::Output::ERROR:
+            return "preprocessorErrorDirective";
+        case simplecpp::Output::SYNTAX_ERROR:
+            return "syntaxError";
+        case simplecpp::Output::UNHANDLED_CHAR_ERROR:
+            return "unhandledChar";
+        case simplecpp::Output::INCLUDE_NESTED_TOO_DEEPLY:
+            return "includeNestedTooDeeply";
+        // should never occur
+        case simplecpp::Output::EXPLICIT_INCLUDE_NOT_FOUND:
+        case simplecpp::Output::FILE_NOT_FOUND:
+        case simplecpp::Output::DUI_ERROR:
+        // handled separately
+        case simplecpp::Output::MISSING_HEADER:
+        // no handled at all (warnings)
+        case simplecpp::Output::WARNING:
+        case simplecpp::Output::PORTABILITY_BACKSLASH:
+            throw std::runtime_error("unexpected simplecpp::Output type");
+    }
+
+    cppcheck::unreachable();
+}
+
+void Preprocessor::error(const std::string &filename, unsigned int linenr, const std::string &msg, simplecpp::Output::Type type)
 {
     std::list<ErrorMessage::FileLocation> locationList;
     if (!filename.empty()) {
@@ -905,7 +931,7 @@ void Preprocessor::error(const std::string &filename, unsigned int linenr, const
                                         mFile0,
                                         Severity::error,
                                         msg,
-                                        "preprocessorErrorDirective",
+                                        simplecppErrToId(type),
                                         Certainty::normal));
 }
 
@@ -935,7 +961,10 @@ void Preprocessor::getErrorMessages(ErrorLogger &errorLogger, const Settings &se
     Preprocessor preprocessor(tokens, settings, errorLogger, Standards::Language::CPP);
     preprocessor.missingInclude("", 1, 2, "", UserHeader);
     preprocessor.missingInclude("", 1, 2, "", SystemHeader);
-    preprocessor.error("", 1, "#error message");   // #error ..
+    preprocessor.error("", 1, "message", simplecpp::Output::ERROR);
+    preprocessor.error("", 1, "message", simplecpp::Output::SYNTAX_ERROR);
+    preprocessor.error("", 1, "message", simplecpp::Output::UNHANDLED_CHAR_ERROR);
+    preprocessor.error("", 1, "message", simplecpp::Output::INCLUDE_NESTED_TOO_DEEPLY);
 }
 
 void Preprocessor::dump(std::ostream &out) const
diff --git a/lib/preprocessor.h b/lib/preprocessor.h
@@ -142,7 +142,7 @@ class CPPCHECKLIB WARN_UNUSED Preprocessor {
 
     bool reportOutput(const simplecpp::OutputList &outputList, bool showerror);
 
-    void error(const std::string &filename, unsigned int linenr, const std::string &msg);
+    void error(const std::string &filename, unsigned int linenr, const std::string &msg, simplecpp::Output::Type type);
 
 private:
     static bool hasErrors(const simplecpp::Output &output);
diff --git a/releasenotes.txt b/releasenotes.txt
@@ -13,6 +13,7 @@ GUI:
 -
 
 Changed interface:
+- some `preprocessorErrorDirective` and `syntaxError` errors got more specific error IDs.
 -
 
 Deprecations:
diff --git a/test/cli/other_test.py b/test/cli/other_test.py
@@ -3890,8 +3890,7 @@ def test_simplecpp_unhandled_char(tmp_path):
     assert stdout.splitlines() == []
     assert stderr.splitlines() == [
         # TODO: lacks column information
-        # TODO: should report another ID
-        '{}:2:0: error: The code contains unhandled character(s) (character code=228). Neither unicode nor extended ascii is supported. [preprocessorErrorDirective]'.format(test_file)
+        '{}:2:0: error: The code contains unhandled character(s) (character code=228). Neither unicode nor extended ascii is supported. [unhandledChar]'.format(test_file)
     ]
 
 
@@ -3922,10 +3921,9 @@ def test_simplecpp_include_nested_too_deeply(tmp_path):
     test_h = tmp_path / 'test_398.h'
     assert stderr.splitlines() == [
         # TODO: should only report the error once
-        # TODO: should report another ID
         # TODO: lacks column information
-        '{}:1:0: error: #include nested too deeply [preprocessorErrorDirective]'.format(test_h),
-        '{}:1:0: error: #include nested too deeply [preprocessorErrorDirective]'.format(test_h)
+        '{}:1:0: error: #include nested too deeply [includeNestedTooDeeply]'.format(test_h),
+        '{}:1:0: error: #include nested too deeply [includeNestedTooDeeply]'.format(test_h)
     ]
 
 
@@ -3946,8 +3944,7 @@ def test_simplecpp_syntax_error(tmp_path):
     assert stdout.splitlines() == []
     assert stderr.splitlines() == [
         # TODO: should only report the error once
-        # TODO: should report another ID
         # TODO: lacks column information
-        '{}:1:0: error: No header in #include [preprocessorErrorDirective]'.format(test_file),
-        '{}:1:0: error: No header in #include [preprocessorErrorDirective]'.format(test_file)
+        '{}:1:0: error: No header in #include [syntaxError]'.format(test_file),
+        '{}:1:0: error: No header in #include [syntaxError]'.format(test_file)
     ]
diff --git a/test/testpreprocessor.cpp b/test/testpreprocessor.cpp
@@ -1529,7 +1529,7 @@ class TestPreprocessor : public TestFixture {
             const std::map<std::string, std::string> actual = getcode(settings0, *this, filedata);
 
             ASSERT_EQUALS(0, actual.size());
-            ASSERT_EQUALS("[file.c:2:0]: (error) No pair for character ('). Can't process file. File is either invalid or unicode, which is currently not supported. [preprocessorErrorDirective]\n", errout_str());
+            ASSERT_EQUALS("[file.c:2:0]: (error) No pair for character ('). Can't process file. File is either invalid or unicode, which is currently not supported. [syntaxError]\n", errout_str());
         }
     }
 
@@ -1544,7 +1544,7 @@ class TestPreprocessor : public TestFixture {
             const std::string actual(expandMacros(filedata, *this));
 
             ASSERT_EQUALS("", actual);
-            ASSERT_EQUALS("[file.cpp:3:0]: (error) No pair for character (\"). Can't process file. File is either invalid or unicode, which is currently not supported. [preprocessorErrorDirective]\n", errout_str());
+            ASSERT_EQUALS("[file.cpp:3:0]: (error) No pair for character (\"). Can't process file. File is either invalid or unicode, which is currently not supported. [syntaxError]\n", errout_str());
         }
 
         {
@@ -1557,7 +1557,7 @@ class TestPreprocessor : public TestFixture {
             const std::string actual(expandMacros(filedata, *this));
 
             ASSERT_EQUALS("", actual);
-            ASSERT_EQUALS("[abc.h:2:0]: (error) No pair for character (\"). Can't process file. File is either invalid or unicode, which is currently not supported. [preprocessorErrorDirective]\n", errout_str());
+            ASSERT_EQUALS("[abc.h:2:0]: (error) No pair for character (\"). Can't process file. File is either invalid or unicode, which is currently not supported. [syntaxError]\n", errout_str());
         }
 
         {
@@ -1570,7 +1570,7 @@ class TestPreprocessor : public TestFixture {
             const std::string actual(expandMacros(filedata, *this));
 
             ASSERT_EQUALS("", actual);
-            ASSERT_EQUALS("[file.cpp:2:0]: (error) No pair for character (\"). Can't process file. File is either invalid or unicode, which is currently not supported. [preprocessorErrorDirective]\n", errout_str());
+            ASSERT_EQUALS("[file.cpp:2:0]: (error) No pair for character (\"). Can't process file. File is either invalid or unicode, which is currently not supported. [syntaxError]\n", errout_str());
         }
 
         {
@@ -1582,7 +1582,7 @@ class TestPreprocessor : public TestFixture {
             const std::string actual(expandMacros(filedata, *this));
 
             ASSERT_EQUALS("", actual);
-            ASSERT_EQUALS("[file.cpp:2:0]: (error) No pair for character (\"). Can't process file. File is either invalid or unicode, which is currently not supported. [preprocessorErrorDirective]\n", errout_str());
+            ASSERT_EQUALS("[file.cpp:2:0]: (error) No pair for character (\"). Can't process file. File is either invalid or unicode, which is currently not supported. [syntaxError]\n", errout_str());
         }
 
         {
@@ -1598,7 +1598,7 @@ class TestPreprocessor : public TestFixture {
             // expand macros..
             (void)expandMacros(filedata, *this);
 
-            ASSERT_EQUALS("[file.cpp:7:0]: (error) No pair for character (\"). Can't process file. File is either invalid or unicode, which is currently not supported. [preprocessorErrorDirective]\n", errout_str());
+            ASSERT_EQUALS("[file.cpp:7:0]: (error) No pair for character (\"). Can't process file. File is either invalid or unicode, which is currently not supported. [syntaxError]\n", errout_str());
         }
     }
 
@@ -1651,7 +1651,7 @@ class TestPreprocessor : public TestFixture {
         // Compare results..
         ASSERT_EQUALS(1, actual.size());
         ASSERT_EQUALS("", actual.at(""));
-        ASSERT_EQUALS("[file.c:6:0]: (error) failed to expand 'BC', Wrong number of parameters for macro 'BC'. [preprocessorErrorDirective]\n", errout_str());
+        ASSERT_EQUALS("[file.c:6:0]: (error) failed to expand 'BC', Wrong number of parameters for macro 'BC'. [syntaxError]\n", errout_str());
     }
 
     void newline_in_macro() {
@@ -1968,12 +1968,12 @@ class TestPreprocessor : public TestFixture {
 
     void invalid_define_1() {
         (void)getcode(settings0, *this, "#define =\n");
-        ASSERT_EQUALS("[file.c:1:0]: (error) Failed to parse #define [preprocessorErrorDirective]\n", errout_str());
+        ASSERT_EQUALS("[file.c:1:0]: (error) Failed to parse #define [syntaxError]\n", errout_str());
     }
 
     void invalid_define_2() {  // #4036
         (void)getcode(settings0, *this, "#define () {(int f(x) }\n");
-        ASSERT_EQUALS("[file.c:1:0]: (error) Failed to parse #define [preprocessorErrorDirective]\n", errout_str());
+        ASSERT_EQUALS("[file.c:1:0]: (error) Failed to parse #define [syntaxError]\n", errout_str());
     }
 
     void inline_suppressions() {
@@ -2119,7 +2119,7 @@ class TestPreprocessor : public TestFixture {
         const char code[] = "#elif (){\n";
         const std::string actual = getcodeforcfg(settings0, *this, code, "TEST", "test.c");
         ASSERT_EQUALS("", actual);
-        ASSERT_EQUALS("[test.c:1:0]: (error) #elif without #if [preprocessorErrorDirective]\n", errout_str());
+        ASSERT_EQUALS("[test.c:1:0]: (error) #elif without #if [syntaxError]\n", errout_str());
     }
 
     void getConfigs1() {
@@ -2368,8 +2368,8 @@ class TestPreprocessor : public TestFixture {
         // Preprocess => don't crash..
         (void)getcode(settings0, *this, filedata);
         ASSERT_EQUALS(
-            "[file.c:1:0]: (error) Syntax error in #ifdef [preprocessorErrorDirective]\n"
-            "[file.c:1:0]: (error) Syntax error in #ifdef [preprocessorErrorDirective]\n", errout_str());
+            "[file.c:1:0]: (error) Syntax error in #ifdef [syntaxError]\n"
+            "[file.c:1:0]: (error) Syntax error in #ifdef [syntaxError]\n", errout_str());
     }
 
     void garbage() {
@@ -2631,7 +2631,7 @@ class TestPreprocessor : public TestFixture {
 
         settings.standards.setStd("c++11");
         ASSERT_EQUALS("", getcodeforcfg(settings, *this, code, "", "test.cpp"));
-        ASSERT_EQUALS("[test.cpp:1:0]: (error) failed to evaluate #if condition, undefined function-like macro invocation: __has_include( ... ) [preprocessorErrorDirective]\n", errout_str());
+        ASSERT_EQUALS("[test.cpp:1:0]: (error) failed to evaluate #if condition, undefined function-like macro invocation: __has_include( ... ) [syntaxError]\n", errout_str()); // TODO: use individual ID
 
         settings.standards.setStd("c++17");
         ASSERT_EQUALS("", getcodeforcfg(settings, *this, code, "", "test.cpp"));
diff --git a/test/testsuppressions.cpp b/test/testsuppressions.cpp
@@ -1355,7 +1355,7 @@ class TestSuppressions : public TestFixture {
                             "[!VAR \"BC\" = \"$BC + 1\"!][!//\n"
                             "[!ENDIF!][!//\n"
                             "};";
-        ASSERT_EQUALS(0, (this->*check)(code, "preprocessorErrorDirective:test.cpp:4"));
+        ASSERT_EQUALS(0, (this->*check)(code, "syntaxError:test.cpp:4"));
         ASSERT_EQUALS("", errout_str());
     }
 

Original file line number	Diff line number	Diff line change
`@@ -1213,7 +1213,7 @@ unsigned int CppCheck::checkInternal(const FileWithDetails& file, const std::str`
`1213`	`1213`
`1214`	`1214`	`if (!hasValidConfig && currCfg == *configurations.rbegin()) {`
`1215`	`1215`	`// If there is no valid configuration then report error..`
`1216`		`- preprocessor.error(o.location.file(), o.location.line, o.msg);`
	`1216`	`+ preprocessor.error(o.location.file(), o.location.line, o.msg, o.type);`
`1217`	`1217`	`}`
`1218`	`1218`	`continue;`
`1219`	`1219`
Original file line number	Diff line number	Diff line change
`@@ -13,6 +13,7 @@ GUI:`
`13`	`13`	`-`
`14`	`14`
`15`	`15`	`Changed interface:`
	`16`	+- some `preprocessorErrorDirective` and `syntaxError` errors got more specific error IDs.
`16`	`17`	`-`
`17`	`18`
`18`	`19`	`Deprecations:`
Original file line number	Diff line number	Diff line change
`@@ -1529,7 +1529,7 @@ class TestPreprocessor : public TestFixture {`
`1529`	`1529`	`const std::map<std::string, std::string> actual = getcode(settings0, *this, filedata);`
`1530`	`1530`
`1531`	`1531`	`ASSERT_EQUALS(0, actual.size());`
`1532`		`- ASSERT_EQUALS("[file.c:2:0]: (error) No pair for character ('). Can't process file. File is either invalid or unicode, which is currently not supported. [preprocessorErrorDirective]\n", errout_str());`
	`1532`	`+ ASSERT_EQUALS("[file.c:2:0]: (error) No pair for character ('). Can't process file. File is either invalid or unicode, which is currently not supported. [syntaxError]\n", errout_str());`
`1533`	`1533`	`}`
`1534`	`1534`	`}`
`1535`	`1535`
`@@ -1544,7 +1544,7 @@ class TestPreprocessor : public TestFixture {`
`1544`	`1544`	`const std::string actual(expandMacros(filedata, *this));`
`1545`	`1545`
`1546`	`1546`	`ASSERT_EQUALS("", actual);`
`1547`		`- ASSERT_EQUALS("[file.cpp:3:0]: (error) No pair for character (\"). Can't process file. File is either invalid or unicode, which is currently not supported. [preprocessorErrorDirective]\n", errout_str());`
	`1547`	`+ ASSERT_EQUALS("[file.cpp:3:0]: (error) No pair for character (\"). Can't process file. File is either invalid or unicode, which is currently not supported. [syntaxError]\n", errout_str());`
`1548`	`1548`	`}`
`1549`	`1549`
`1550`	`1550`	`{`
`@@ -1557,7 +1557,7 @@ class TestPreprocessor : public TestFixture {`
`1557`	`1557`	`const std::string actual(expandMacros(filedata, *this));`
`1558`	`1558`
`1559`	`1559`	`ASSERT_EQUALS("", actual);`
`1560`		`- ASSERT_EQUALS("[abc.h:2:0]: (error) No pair for character (\"). Can't process file. File is either invalid or unicode, which is currently not supported. [preprocessorErrorDirective]\n", errout_str());`
	`1560`	`+ ASSERT_EQUALS("[abc.h:2:0]: (error) No pair for character (\"). Can't process file. File is either invalid or unicode, which is currently not supported. [syntaxError]\n", errout_str());`
`1561`	`1561`	`}`
`1562`	`1562`
`1563`	`1563`	`{`
`@@ -1570,7 +1570,7 @@ class TestPreprocessor : public TestFixture {`
`1570`	`1570`	`const std::string actual(expandMacros(filedata, *this));`
`1571`	`1571`
`1572`	`1572`	`ASSERT_EQUALS("", actual);`
`1573`		`- ASSERT_EQUALS("[file.cpp:2:0]: (error) No pair for character (\"). Can't process file. File is either invalid or unicode, which is currently not supported. [preprocessorErrorDirective]\n", errout_str());`
	`1573`	`+ ASSERT_EQUALS("[file.cpp:2:0]: (error) No pair for character (\"). Can't process file. File is either invalid or unicode, which is currently not supported. [syntaxError]\n", errout_str());`
`1574`	`1574`	`}`
`1575`	`1575`
`1576`	`1576`	`{`
`@@ -1582,7 +1582,7 @@ class TestPreprocessor : public TestFixture {`
`1582`	`1582`	`const std::string actual(expandMacros(filedata, *this));`
`1583`	`1583`
`1584`	`1584`	`ASSERT_EQUALS("", actual);`
`1585`		`- ASSERT_EQUALS("[file.cpp:2:0]: (error) No pair for character (\"). Can't process file. File is either invalid or unicode, which is currently not supported. [preprocessorErrorDirective]\n", errout_str());`
	`1585`	`+ ASSERT_EQUALS("[file.cpp:2:0]: (error) No pair for character (\"). Can't process file. File is either invalid or unicode, which is currently not supported. [syntaxError]\n", errout_str());`
`1586`	`1586`	`}`
`1587`	`1587`
`1588`	`1588`	`{`
`@@ -1598,7 +1598,7 @@ class TestPreprocessor : public TestFixture {`
`1598`	`1598`	`// expand macros..`
`1599`	`1599`	`(void)expandMacros(filedata, *this);`
`1600`	`1600`
`1601`		`- ASSERT_EQUALS("[file.cpp:7:0]: (error) No pair for character (\"). Can't process file. File is either invalid or unicode, which is currently not supported. [preprocessorErrorDirective]\n", errout_str());`
	`1601`	`+ ASSERT_EQUALS("[file.cpp:7:0]: (error) No pair for character (\"). Can't process file. File is either invalid or unicode, which is currently not supported. [syntaxError]\n", errout_str());`
`1602`	`1602`	`}`
`1603`	`1603`	`}`
`1604`	`1604`
`@@ -1651,7 +1651,7 @@ class TestPreprocessor : public TestFixture {`
`1651`	`1651`	`// Compare results..`
`1652`	`1652`	`ASSERT_EQUALS(1, actual.size());`
`1653`	`1653`	`ASSERT_EQUALS("", actual.at(""));`
`1654`		`- ASSERT_EQUALS("[file.c:6:0]: (error) failed to expand 'BC', Wrong number of parameters for macro 'BC'. [preprocessorErrorDirective]\n", errout_str());`
	`1654`	`+ ASSERT_EQUALS("[file.c:6:0]: (error) failed to expand 'BC', Wrong number of parameters for macro 'BC'. [syntaxError]\n", errout_str());`
`1655`	`1655`	`}`
`1656`	`1656`
`1657`	`1657`	`void newline_in_macro() {`
`@@ -1968,12 +1968,12 @@ class TestPreprocessor : public TestFixture {`
`1968`	`1968`
`1969`	`1969`	`void invalid_define_1() {`
`1970`	`1970`	`(void)getcode(settings0, *this, "#define =\n");`
`1971`		`- ASSERT_EQUALS("[file.c:1:0]: (error) Failed to parse #define [preprocessorErrorDirective]\n", errout_str());`
	`1971`	`+ ASSERT_EQUALS("[file.c:1:0]: (error) Failed to parse #define [syntaxError]\n", errout_str());`
`1972`	`1972`	`}`
`1973`	`1973`
`1974`	`1974`	`void invalid_define_2() { // #4036`
`1975`	`1975`	`(void)getcode(settings0, *this, "#define () {(int f(x) }\n");`
`1976`		`- ASSERT_EQUALS("[file.c:1:0]: (error) Failed to parse #define [preprocessorErrorDirective]\n", errout_str());`
	`1976`	`+ ASSERT_EQUALS("[file.c:1:0]: (error) Failed to parse #define [syntaxError]\n", errout_str());`
`1977`	`1977`	`}`
`1978`	`1978`
`1979`	`1979`	`void inline_suppressions() {`
`@@ -2119,7 +2119,7 @@ class TestPreprocessor : public TestFixture {`
`2119`	`2119`	`const char code[] = "#elif (){\n";`
`2120`	`2120`	`const std::string actual = getcodeforcfg(settings0, *this, code, "TEST", "test.c");`
`2121`	`2121`	`ASSERT_EQUALS("", actual);`
`2122`		`- ASSERT_EQUALS("[test.c:1:0]: (error) #elif without #if [preprocessorErrorDirective]\n", errout_str());`
	`2122`	`+ ASSERT_EQUALS("[test.c:1:0]: (error) #elif without #if [syntaxError]\n", errout_str());`
`2123`	`2123`	`}`
`2124`	`2124`
`2125`	`2125`	`void getConfigs1() {`
`@@ -2368,8 +2368,8 @@ class TestPreprocessor : public TestFixture {`
`2368`	`2368`	`// Preprocess => don't crash..`
`2369`	`2369`	`(void)getcode(settings0, *this, filedata);`
`2370`	`2370`	`ASSERT_EQUALS(`
`2371`		`- "[file.c:1:0]: (error) Syntax error in #ifdef [preprocessorErrorDirective]\n"`
`2372`		`- "[file.c:1:0]: (error) Syntax error in #ifdef [preprocessorErrorDirective]\n", errout_str());`
	`2371`	`+ "[file.c:1:0]: (error) Syntax error in #ifdef [syntaxError]\n"`
	`2372`	`+ "[file.c:1:0]: (error) Syntax error in #ifdef [syntaxError]\n", errout_str());`
`2373`	`2373`	`}`
`2374`	`2374`
`2375`	`2375`	`void garbage() {`
`@@ -2631,7 +2631,7 @@ class TestPreprocessor : public TestFixture {`
`2631`	`2631`
`2632`	`2632`	`settings.standards.setStd("c++11");`
`2633`	`2633`	`ASSERT_EQUALS("", getcodeforcfg(settings, *this, code, "", "test.cpp"));`
`2634`		`- ASSERT_EQUALS("[test.cpp:1:0]: (error) failed to evaluate #if condition, undefined function-like macro invocation: __has_include( ... ) [preprocessorErrorDirective]\n", errout_str());`
	`2634`	`+ ASSERT_EQUALS("[test.cpp:1:0]: (error) failed to evaluate #if condition, undefined function-like macro invocation: __has_include( ... ) [syntaxError]\n", errout_str()); // TODO: use individual ID`
`2635`	`2635`
`2636`	`2636`	`settings.standards.setStd("c++17");`
`2637`	`2637`	`ASSERT_EQUALS("", getcodeforcfg(settings, *this, code, "", "test.cpp"));`
Original file line number	Diff line number	Diff line change
`@@ -1355,7 +1355,7 @@ class TestSuppressions : public TestFixture {`
`1355`	`1355`	`"[!VAR \"BC\" = \"$BC + 1\"!][!//\n"`
`1356`	`1356`	`"[!ENDIF!][!//\n"`
`1357`	`1357`	`"};";`
`1358`		`- ASSERT_EQUALS(0, (this->*check)(code, "preprocessorErrorDirective:test.cpp:4"));`
	`1358`	`+ ASSERT_EQUALS(0, (this->*check)(code, "syntaxError:test.cpp:4"));`
`1359`	`1359`	`ASSERT_EQUALS("", errout_str());`
`1360`	`1360`	`}`
`1361`	`1361`