fixed unmatchedSuppression selfcheck warnings

Author: firewave

.selfcheck_suppressions

@@ -2,7 +2,6 @@ missingIncludeSystem
 
 # temporary suppressions - fix the warnings!
 simplifyUsing:lib/valueptr.h
-varid0:gui/projectfile.cpp
 naming-privateMemberVariable:gui/test/cppchecklibrarydata/testcppchecklibrarydata.h
 symbolDatabaseWarning:*/moc_*.cpp
 simplifyUsing:*/moc_*.cpp

lib/checkclass.cpp

@@ -3052,7 +3052,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

@@ -153,7 +153,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;
@@ -169,7 +168,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;
@@ -190,7 +188,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;
@@ -207,7 +204,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

@@ -791,7 +791,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(),
@@ -7886,7 +7885,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;