Skip to content

Commit 9a69950

Browse files
committed
CheckUnusedFunctions: added dedicated parseTokens()
1 parent dc094c3 commit 9a69950

3 files changed

Lines changed: 10 additions & 3 deletions

File tree

lib/checkunusedfunctions.cpp

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -370,12 +370,17 @@ void CheckUnusedFunctions::unusedFunctionError(ErrorLogger * const errorLogger,
370370
Check::writeToErrorList(errmsg);
371371
}
372372

373-
Check::FileInfo *CheckUnusedFunctions::getFileInfo(const Tokenizer *tokenizer, const Settings *settings) const
373+
void CheckUnusedFunctions::parseTokens(const Tokenizer *tokenizer, const Settings *settings)
374374
{
375375
if (!settings->checks.isEnabled(Checks::unusedFunction))
376-
return nullptr;
376+
return;
377377
if (settings->useSingleJob() && settings->buildDir.empty())
378378
instance.parseTokens(*tokenizer, tokenizer->list.getFiles().front().c_str(), settings);
379+
}
380+
381+
Check::FileInfo *CheckUnusedFunctions::getFileInfo(const Tokenizer *tokenizer, const Settings *settings) const
382+
{
383+
parseTokens(tokenizer, settings);
379384
return nullptr;
380385
}
381386

lib/checkunusedfunctions.h

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -62,6 +62,8 @@ class CPPCHECKLIB CheckUnusedFunctions : public Check {
6262
// * What functions are declared
6363
void parseTokens(const Tokenizer &tokenizer, const char FileName[], const Settings *settings);
6464

65+
static void parseTokens(const Tokenizer *tokenizer, const Settings *settings);
66+
6567
// Return true if an error is reported.
6668
bool check(ErrorLogger * const errorLogger, const Settings& settings) const;
6769

lib/cppcheck.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -640,7 +640,7 @@ unsigned int CppCheck::checkFile(const std::string& filename, const std::string
640640
std::ifstream in(filename);
641641
tokenizer.list.createTokens(in, filename);
642642
}
643-
checkUnusedFunctions.getFileInfo(&tokenizer, &mSettings);
643+
CheckUnusedFunctions::parseTokens(&tokenizer, &mSettings);
644644
return EXIT_SUCCESS;
645645
}
646646

0 commit comments

Comments
 (0)