Executor: adjusted some checks in hasToLog() (#7210)

firewave · web-flow · commit 784401e839ef · 2025-02-05T15:03:15.000+01:00
diff --git a/cli/executor.cpp b/cli/executor.cpp
@@ -40,8 +40,8 @@ Executor::Executor(const std::list<FileWithDetails> &files, const std::list<File
 // TODO: this logic is duplicated in CppCheck::reportErr()
 bool Executor::hasToLog(const ErrorMessage &msg)
 {
-    if (!mSettings.library.reportErrors(msg.file0))
-        return false;
+    if (msg.severity == Severity::internal)
+        return true;
 
     if (!mSuppressions.isSuppressed(msg, {}))
     {

Original file line number	Diff line number	Diff line change
`@@ -40,8 +40,8 @@ Executor::Executor(const std::list<FileWithDetails> &files, const std::list<File`
`40`	`40`	`// TODO: this logic is duplicated in CppCheck::reportErr()`
`41`	`41`	`bool Executor::hasToLog(const ErrorMessage &msg)`
`42`	`42`	`{`
`43`		`- if (!mSettings.library.reportErrors(msg.file0))`
`44`		`- return false;`
	`43`	`+ if (msg.severity == Severity::internal)`
	`44`	`+ return true;`
`45`	`45`
`46`	`46`	`if (!mSuppressions.isSuppressed(msg, {}))`
`47`	`47`	`{`