Skip to content

Commit a901cd7

Browse files
committed
fixed unmatchedSuppression selfcheck warnings
1 parent ca6b2ab commit a901cd7

File tree

4 files changed

+1
-7
lines changed

4 files changed

+1
-7
lines changed

.selfcheck_suppressions

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,6 @@ missingIncludeSystem
22

33
# temporary suppressions - fix the warnings!
44
simplifyUsing:lib/valueptr.h
5-
varid0:gui/projectfile.cpp
65
naming-privateMemberVariable:gui/test/cppchecklibrarydata/testcppchecklibrarydata.h
76
symbolDatabaseWarning:*/moc_*.cpp
87
simplifyUsing:*/moc_*.cpp

lib/checkclass.cpp

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3095,7 +3095,6 @@ void CheckClass::checkCopyCtorAndEqOperator()
30953095
{
30963096
// This is disabled because of #8388
30973097
// The message must be clarified. How is the behaviour different?
3098-
// cppcheck-suppress unreachableCode - remove when code is enabled again
30993098
if ((true) || !mSettings->severity.isEnabled(Severity::warning)) // NOLINT(readability-simplify-boolean-expr)
31003099
return;
31013100

lib/keywords.cpp

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -169,7 +169,6 @@ static const std::unordered_set<std::string> cpp26_keywords_all = {
169169
// cppcheck-suppress unusedFunction
170170
const std::unordered_set<std::string>& Keywords::getAll(Standards::cstd_t cStd)
171171
{
172-
// cppcheck-suppress missingReturn
173172
switch (cStd) {
174173
case Standards::cstd_t::C89:
175174
return c89_keywords_all;
@@ -187,7 +186,6 @@ const std::unordered_set<std::string>& Keywords::getAll(Standards::cstd_t cStd)
187186

188187
// cppcheck-suppress unusedFunction
189188
const std::unordered_set<std::string>& Keywords::getAll(Standards::cppstd_t cppStd) {
190-
// cppcheck-suppress missingReturn
191189
switch (cppStd) {
192190
case Standards::cppstd_t::CPP03:
193191
return cpp03_keywords_all;
@@ -210,7 +208,6 @@ const std::unordered_set<std::string>& Keywords::getAll(Standards::cppstd_t cppS
210208
// cppcheck-suppress unusedFunction
211209
const std::unordered_set<std::string>& Keywords::getOnly(Standards::cstd_t cStd)
212210
{
213-
// cppcheck-suppress missingReturn
214211
switch (cStd) {
215212
case Standards::cstd_t::C89:
216213
return c89_keywords;
@@ -229,7 +226,6 @@ const std::unordered_set<std::string>& Keywords::getOnly(Standards::cstd_t cStd)
229226
// cppcheck-suppress unusedFunction
230227
const std::unordered_set<std::string>& Keywords::getOnly(Standards::cppstd_t cppStd)
231228
{
232-
// cppcheck-suppress missingReturn
233229
switch (cppStd) {
234230
case Standards::cppstd_t::CPP03:
235231
return cpp03_keywords;

lib/valueflow.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4831,7 +4831,7 @@ static void addToErrorPath(ValueFlow::Value& value, const ValueFlow::Value& from
48314831
}
48324832

48334833
static std::vector<Token*> findAllUsages(const Variable* var,
4834-
Token* start, // cppcheck-suppress constParameterPointer // FP
4834+
Token* start,
48354835
const Library& library)
48364836
{
48374837
// std::vector<Token*> result;

0 commit comments

Comments
 (0)