fixed unmatchedSuppression selfcheck warnings

Author: firewave

lib/checkclass.cpp

@@ -3040,7 +3040,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;
 

lib/keywords.cpp

@@ -155,7 +155,6 @@ static const std::unordered_set<std::string> cpp23_keywords_all = cpp20_keywords
 // cppcheck-suppress unusedFunction
 const std::unordered_set<std::string>& Keywords::getAll(Standards::cstd_t cStd)
 {
-    // cppcheck-suppress missingReturn
     switch (cStd) {
     case Standards::cstd_t::C89:
         return c89_keywords_all;
@@ -171,7 +170,6 @@ const std::unordered_set<std::string>& Keywords::getAll(Standards::cstd_t cStd)
 
 // cppcheck-suppress unusedFunction
 const std::unordered_set<std::string>& Keywords::getAll(Standards::cppstd_t cppStd) {
-    // cppcheck-suppress missingReturn
     switch (cppStd) {
     case Standards::cppstd_t::CPP03:
         return cpp03_keywords_all;
@@ -192,7 +190,6 @@ const std::unordered_set<std::string>& Keywords::getAll(Standards::cppstd_t cppS
 // cppcheck-suppress unusedFunction
 const std::unordered_set<std::string>& Keywords::getOnly(Standards::cstd_t cStd)
 {
-    // cppcheck-suppress missingReturn
     switch (cStd) {
     case Standards::cstd_t::C89:
         return c89_keywords;
@@ -209,7 +206,6 @@ const std::unordered_set<std::string>& Keywords::getOnly(Standards::cstd_t cStd)
 // cppcheck-suppress unusedFunction
 const std::unordered_set<std::string>& Keywords::getOnly(Standards::cppstd_t cppStd)
 {
-    // cppcheck-suppress missingReturn
     switch (cppStd) {
     case Standards::cppstd_t::CPP03:
         return cpp03_keywords;

lib/valueflow.cpp

@@ -790,7 +790,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(),
@@ -7803,7 +7802,7 @@ static void addToErrorPath(ValueFlow::Value& value, const ValueFlow::Value& from
 }
 
 static std::vector<Token*> findAllUsages(const Variable* var,
-                                         Token* start, // cppcheck-suppress constParameterPointer // FP
+                                         Token* start,
                                          const Library* library)
 {
     // std::vector<Token*> result;