fixed unmatchedSuppression selfcheck warnings

firewave · firewave · commit 34b87f6e8370 · 2023-09-07T12:16:47.000+02:00
diff --git a/lib/checkclass.cpp b/lib/checkclass.cpp
@@ -3041,7 +3041,6 @@ void CheckClass::checkCopyCtorAndEqOperator()
 {
     // This is disabled because of #8388
     // The message must be clarified. How is the behaviour different?
-    // cppcheck-suppress unreachableCode - remove when code is enabled again
     if ((true) || !mSettings->severity.isEnabled(Severity::warning)) // NOLINT(readability-simplify-boolean-expr)
         return;
 
diff --git a/lib/valueflow.cpp b/lib/valueflow.cpp
@@ -789,7 +789,6 @@ static void setTokenValue(Token* tok,
             }
         } else if (!value.isImpossible()) {
             // is condition only depending on 1 variable?
-            // cppcheck-suppress[variableScope] #8541
             nonneg int varId = 0;
             bool ret = false;
             visitAstNodes(parent->astOperand1(),

Original file line number	Diff line number	Diff line change
`@@ -3041,7 +3041,6 @@ void CheckClass::checkCopyCtorAndEqOperator()`
`3041`	`3041`	`{`
`3042`	`3042`	`// This is disabled because of #8388`
`3043`	`3043`	`// The message must be clarified. How is the behaviour different?`
`3044`		`- // cppcheck-suppress unreachableCode - remove when code is enabled again`
`3045`	`3044`	`if ((true) \|\| !mSettings->severity.isEnabled(Severity::warning)) // NOLINT(readability-simplify-boolean-expr)`
`3046`	`3045`	`return;`
`3047`	`3046`
Original file line number	Diff line number	Diff line change
`@@ -789,7 +789,6 @@ static void setTokenValue(Token* tok,`
`789`	`789`	`}`
`790`	`790`	`} else if (!value.isImpossible()) {`
`791`	`791`	`// is condition only depending on 1 variable?`
`792`		`- // cppcheck-suppress[variableScope] #8541`
`793`	`792`	`nonneg int varId = 0;`
`794`	`793`	`bool ret = false;`
`795`	`794`	`visitAstNodes(parent->astOperand1(),`