Skip to content

Commit 1cf85c0

Browse files
firewavefirewave
committed
reduced Tokenizer::isC() usage [skip ci]
1 parent da5006a commit 1cf85c0

File tree

10 files changed

+24
-18
lines changed

10 files changed

+24
-18
lines changed

‎lib/checkautovariables.cpp‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -241,10 +241,10 @@ static bool isAutoVariableRHS(const Token* tok) {
241241
return isAddressOfLocalVariable(tok) || isAutoVarArray(tok) || isLocalContainerBuffer(tok);
242242
}
243243

244-
static bool hasOverloadedAssignment(const Token* tok, bool c, bool& inconclusive)
244+
static bool hasOverloadedAssignment(const Token* tok, bool& inconclusive)
245245
{
246246
inconclusive = false;
247-
if (c)
247+
if (tok->isC())
248248
return false;
249249
if (const ValueType* vt = tok->valueType()) {
250250
if (vt->pointer && !Token::simpleMatch(tok->astParent(), "*"))
@@ -280,13 +280,13 @@ void CheckAutoVariables::autoVariables()
280280
} else if (Token::Match(tok, "[;{}] * %var% =") && isPtrArg(tok->tokAt(2)) && isAutoVariableRHS(tok->tokAt(3)->astOperand2())) {
281281
const Token* lhs = tok->tokAt(2);
282282
bool inconclusive = false;
283-
if (!hasOverloadedAssignment(lhs, mTokenizer->isC(), inconclusive) || (printInconclusive && inconclusive))
283+
if (!hasOverloadedAssignment(lhs, inconclusive) || (printInconclusive && inconclusive))
284284
checkAutoVariableAssignment(tok->next(), inconclusive);
285285
tok = tok->tokAt(4);
286286
} else if (Token::Match(tok, "[;{}] %var% . %var% =") && isPtrArg(tok->next()) && isAutoVariableRHS(tok->tokAt(4)->astOperand2())) {
287287
const Token* lhs = tok->tokAt(3);
288288
bool inconclusive = false;
289-
if (!hasOverloadedAssignment(lhs, mTokenizer->isC(), inconclusive) || (printInconclusive && inconclusive))
289+
if (!hasOverloadedAssignment(lhs, inconclusive) || (printInconclusive && inconclusive))
290290
checkAutoVariableAssignment(tok->next(), inconclusive);
291291
tok = tok->tokAt(5);
292292
} else if (Token::Match(tok, "[;{}] %var% [") && Token::simpleMatch(tok->linkAt(2), "] =") &&

‎lib/checkcondition.cpp‎

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1419,8 +1419,6 @@ void CheckCondition::clarifyCondition()
14191419
if (!mSettings->severity.isEnabled(Severity::style) && !mSettings->isPremiumEnabled("clarifyCondition"))
14201420
return;
14211421

1422-
const bool isC = mTokenizer->isC();
1423-
14241422
logChecker("CheckCondition::clarifyCondition"); // style
14251423

14261424
const SymbolDatabase *symbolDatabase = mTokenizer->getSymbolDatabase();
@@ -1432,7 +1430,7 @@ void CheckCondition::clarifyCondition()
14321430
tok2 = tok2->link();
14331431
else if (tok2->isComparisonOp()) {
14341432
// This might be a template
1435-
if (!isC && tok2->link())
1433+
if (!tok2->isC() && tok2->link())
14361434
break;
14371435
if (Token::simpleMatch(tok2->astParent(), "?"))
14381436
break;

‎lib/checkfunctions.cpp‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -56,7 +56,7 @@ static const CWE CWE688(688U); // Function Call With Incorrect Variable or Refe
5656

5757
void CheckFunctions::checkProhibitedFunctions()
5858
{
59-
const bool checkAlloca = mSettings->severity.isEnabled(Severity::warning) && ((mSettings->standards.c >= Standards::C99 && mTokenizer->isC()) || mSettings->standards.cpp >= Standards::CPP11);
59+
const bool checkAlloca = mSettings->severity.isEnabled(Severity::warning) && ((mTokenizer->isC() && mSettings->standards.c >= Standards::C99) || mSettings->standards.cpp >= Standards::CPP11);
6060

6161
logChecker("CheckFunctions::checkProhibitedFunctions");
6262

@@ -67,7 +67,7 @@ void CheckFunctions::checkProhibitedFunctions()
6767
continue;
6868
// alloca() is special as it depends on the code being C or C++, so it is not in Library
6969
if (checkAlloca && Token::simpleMatch(tok, "alloca (") && (!tok->function() || tok->function()->nestedIn->type == Scope::eGlobal)) {
70-
if (mTokenizer->isC()) {
70+
if (tok->isC()) {
7171
if (mSettings->standards.c > Standards::C89)
7272
reportError(tok, Severity::warning, "allocaCalled",
7373
"$symbol:alloca\n"
@@ -315,7 +315,7 @@ void CheckFunctions::checkMissingReturn()
315315
const Function *function = scope->function;
316316
if (!function || !function->hasBody())
317317
continue;
318-
if (function->name() == "main" && !(mSettings->standards.c < Standards::C99 && mTokenizer->isC()))
318+
if (function->name() == "main" && !(mTokenizer->isC() && mSettings->standards.c < Standards::C99))
319319
continue;
320320
if (function->type != Function::Type::eFunction && function->type != Function::Type::eOperatorEqual)
321321
continue;

‎lib/checkmemoryleak.cpp‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -375,7 +375,7 @@ CheckMemoryLeak::AllocType CheckMemoryLeak::functionReturnType(const Function* f
375375
if (Token::Match(tok, "= %varid% ;", varid)) {
376376
return No;
377377
}
378-
if (!mTokenizer_->isC() && Token::Match(tok, "[(,] %varid% [,)]", varid)) {
378+
if (!tok->isC() && Token::Match(tok, "[(,] %varid% [,)]", varid)) {
379379
return No;
380380
}
381381
if (Token::Match(tok, "[(,] & %varid% [.,)]", varid)) {

‎lib/checkother.cpp‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -938,7 +938,7 @@ void CheckOther::checkVariableScope()
938938
continue;
939939

940940
const bool isPtrOrRef = var->isPointer() || var->isReference();
941-
const bool isSimpleType = var->typeStartToken()->isStandardType() || var->typeStartToken()->isEnumType() || (mTokenizer->isC() && var->type() && var->type()->isStructType());
941+
const bool isSimpleType = var->typeStartToken()->isStandardType() || var->typeStartToken()->isEnumType() || (var->typeStartToken()->isC() && var->type() && var->type()->isStructType());
942942
if (!isPtrOrRef && !isSimpleType && !astIsContainer(var->nameToken()))
943943
continue;
944944

‎lib/checkstring.cpp‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -193,7 +193,7 @@ void CheckString::checkSuspiciousStringCompare()
193193
continue;
194194

195195
if (litTok->tokType() == Token::eString) {
196-
if (mTokenizer->isC() || (varType && varType->pointer))
196+
if (varTok->isC() || (varType && varType->pointer))
197197
suspiciousStringCompareError(tok, varTok->expressionString(), litTok->isLong());
198198
} else if (litTok->tokType() == Token::eChar && varType && varType->pointer) {
199199
suspiciousStringCompareError_char(tok, varTok->expressionString());

‎lib/checkuninitvar.cpp‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -166,7 +166,7 @@ void CheckUninitVar::checkScope(const Scope* scope, const std::set<std::string>
166166
continue;
167167
}
168168

169-
bool stdtype = mTokenizer->isC() && arrayTypeDefs.find(var.typeStartToken()->str()) == arrayTypeDefs.end();
169+
bool stdtype = var.typeStartToken()->isC() && arrayTypeDefs.find(var.typeStartToken()->str()) == arrayTypeDefs.end();
170170
const Token* tok = var.typeStartToken();
171171
for (; tok != var.nameToken() && tok->str() != "<"; tok = tok->next()) {
172172
if (tok->isStandardType() || tok->isEnumType())

‎lib/symboldatabase.cpp‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -167,7 +167,7 @@ void SymbolDatabase::createSymbolDatabaseFindAllScopes()
167167
!Token::Match(tok->previous(), "new|friend|const|enum|typedef|mutable|volatile|using|)|(|<")) ||
168168
(Token::Match(tok, "enum class| %name% {") ||
169169
Token::Match(tok, "enum class| %name% : %name% {"))))
170-
|| (mTokenizer.isC() && tok->isKeyword() && Token::Match(tok, "struct|union|enum %name% {"))) {
170+
|| (tok->isC() && tok->isKeyword() && Token::Match(tok, "struct|union|enum %name% {"))) {
171171
const Token *tok2 = tok->tokAt(2);
172172

173173
if (tok->strAt(1) == "::")
@@ -2113,7 +2113,7 @@ bool SymbolDatabase::isFunction(const Token *tok, const Scope* outerScope, const
21132113
else if (Token::Match(tok, "%name% (") && !isReservedName(tok->str()) &&
21142114
Token::simpleMatch(tok->linkAt(1), ") {") &&
21152115
(!tok->previous() || Token::Match(tok->previous(), ";|}"))) {
2116-
if (mTokenizer.isC()) {
2116+
if (tok->isC()) {
21172117
returnImplicitIntError(tok);
21182118
*funcStart = tok;
21192119
*argStart = tok->next();
@@ -6242,7 +6242,7 @@ const Type* SymbolDatabase::findType(const Token *startTok, const Scope *startSc
62426242
if (startTok->str() == startScope->className && startScope->isClassOrStruct() && startTok->strAt(1) != "::")
62436243
return startScope->definedType;
62446244

6245-
if (mTokenizer.isC()) {
6245+
if (startTok->isC()) {
62466246
const Scope* scope = startScope;
62476247
while (scope) {
62486248
if (startTok->str() == scope->className && scope->isClassOrStruct())

‎lib/token.cpp‎

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2747,3 +2747,8 @@ bool Token::isCpp() const
27472747
{
27482748
return mTokensFrontBack.list.isCPP();
27492749
}
2750+
2751+
bool Token::isC() const
2752+
{
2753+
return mTokensFrontBack.list.isC();
2754+
}

‎lib/token.h‎

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1479,8 +1479,11 @@ class CPPCHECKLIB Token {
14791479
mImpl->mDebug = td;
14801480
}
14811481

1482-
/** defaults to C++ if it cannot be determined */
1482+
/** defaults to true if it cannot be determined */
14831483
bool isCpp() const;
1484+
1485+
/** defaults to false if it cannot be determined */
1486+
bool isC() const;
14841487
};
14851488

14861489
Token* findTypeEnd(Token* tok);

0 commit comments

Comments
 (0)