268 lines
10 KiB
C++
268 lines
10 KiB
C++
/*
|
|
* Cppcheck - A tool for static C/C++ code analysis
|
|
* Copyright (C) 2007-2014 Daniel Marjamäki and Cppcheck team.
|
|
*
|
|
* This program is free software: you can redistribute it and/or modify
|
|
* it under the terms of the GNU General Public License as published by
|
|
* the Free Software Foundation, either version 3 of the License, or
|
|
* (at your option) any later version.
|
|
*
|
|
* This program is distributed in the hope that it will be useful,
|
|
* but WITHOUT ANY WARRANTY; without even the implied warranty of
|
|
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
|
|
* GNU General Public License for more details.
|
|
*
|
|
* You should have received a copy of the GNU General Public License
|
|
* along with this program. If not, see <http://www.gnu.org/licenses/>.
|
|
*/
|
|
|
|
|
|
//---------------------------------------------------------------------------
|
|
#include "checkunusedfunctions.h"
|
|
#include "tokenize.h"
|
|
#include "token.h"
|
|
#include "symboldatabase.h"
|
|
#include <cctype>
|
|
//---------------------------------------------------------------------------
|
|
|
|
|
|
|
|
// Register this check class
|
|
CheckUnusedFunctions CheckUnusedFunctions::instance;
|
|
|
|
|
|
//---------------------------------------------------------------------------
|
|
// FUNCTION USAGE - Check for unused functions etc
|
|
//---------------------------------------------------------------------------
|
|
|
|
void CheckUnusedFunctions::parseTokens(const Tokenizer &tokenizer, const char FileName[], const Settings *settings)
|
|
{
|
|
const SymbolDatabase* symbolDatabase = tokenizer.getSymbolDatabase();
|
|
|
|
// Function declarations..
|
|
for (std::size_t i = 0; i < symbolDatabase->functionScopes.size(); i++) {
|
|
const Scope* scope = symbolDatabase->functionScopes[i];
|
|
const Function* func = scope->function;
|
|
if (!func || !func->token || scope->classStart->fileIndex() != 0)
|
|
continue;
|
|
|
|
// Don't warn about functions that are marked by __attribute__((constructor)) or __attribute__((destructor))
|
|
if (func->isAttributeConstructor() || func->isAttributeDestructor() || func->type != Function::eFunction)
|
|
continue;
|
|
|
|
// Don't care about templates
|
|
if (func->retDef->str() == "template")
|
|
continue;
|
|
|
|
FunctionUsage &usage = _functions[func->name()];
|
|
|
|
if (!usage.lineNumber)
|
|
usage.lineNumber = func->token->linenr();
|
|
|
|
// No filename set yet..
|
|
if (usage.filename.empty()) {
|
|
usage.filename = tokenizer.list.getSourceFilePath();
|
|
}
|
|
// Multiple files => filename = "+"
|
|
else if (usage.filename != tokenizer.list.getSourceFilePath()) {
|
|
//func.filename = "+";
|
|
usage.usedOtherFile |= usage.usedSameFile;
|
|
}
|
|
}
|
|
|
|
// Function usage..
|
|
for (const Token *tok = tokenizer.tokens(); tok; tok = tok->next()) {
|
|
|
|
// parsing of library code to find called functions
|
|
if (settings->library.isexecutableblock(FileName, tok->str())) {
|
|
const Token * markupVarToken = tok->tokAt(settings->library.blockstartoffset(FileName));
|
|
int scope = 0;
|
|
bool start = true;
|
|
// find all function calls in library code (starts with '(', not if or while etc)
|
|
while (scope || start) {
|
|
if (markupVarToken->str() == settings->library.blockstart(FileName)) {
|
|
scope++;
|
|
if (start) {
|
|
start = false;
|
|
}
|
|
} else if (markupVarToken->str() == settings->library.blockend(FileName))
|
|
scope--;
|
|
else if (!settings->library.iskeyword(FileName, markupVarToken->str())) {
|
|
if (_functions.find(markupVarToken->str()) != _functions.end())
|
|
_functions[markupVarToken->str()].usedOtherFile = true;
|
|
else if (markupVarToken->next()->str() == "(") {
|
|
FunctionUsage &func = _functions[markupVarToken->str()];
|
|
func.filename = tokenizer.list.getSourceFilePath();
|
|
if (func.filename.empty() || func.filename == "+")
|
|
func.usedOtherFile = true;
|
|
else
|
|
func.usedSameFile = true;
|
|
}
|
|
}
|
|
markupVarToken = markupVarToken->next();
|
|
}
|
|
}
|
|
|
|
if (!settings->library.markupFile(FileName) // only check source files
|
|
&& settings->library.isexporter(tok->str()) && tok->next() != 0) {
|
|
const Token * qPropToken = tok;
|
|
qPropToken = qPropToken->next();
|
|
while (qPropToken && qPropToken->str() != ")") {
|
|
if (settings->library.isexportedprefix(tok->str(), qPropToken->str())) {
|
|
const Token* qNextPropToken = qPropToken->next();
|
|
const std::string& value = qNextPropToken->str();
|
|
if (_functions.find(value) != _functions.end()) {
|
|
_functions[value].usedOtherFile = true;
|
|
}
|
|
}
|
|
if (settings->library.isexportedsuffix(tok->str(), qPropToken->str())) {
|
|
const Token* qNextPropToken = qPropToken->previous();
|
|
const std::string& value = qNextPropToken->str();
|
|
if (value != ")" && _functions.find(value) != _functions.end()) {
|
|
_functions[value].usedOtherFile = true;
|
|
}
|
|
}
|
|
qPropToken = qPropToken->next();
|
|
}
|
|
}
|
|
|
|
if (settings->library.markupFile(FileName)
|
|
&& settings->library.isimporter(FileName, tok->str()) && tok->next()) {
|
|
const Token * qPropToken = tok;
|
|
qPropToken = qPropToken->next();
|
|
if (qPropToken->next()) {
|
|
qPropToken = qPropToken->next();
|
|
while (qPropToken && qPropToken->str() != ")") {
|
|
const std::string& value = qPropToken->str();
|
|
if (!value.empty()) {
|
|
_functions[value].usedOtherFile = true;
|
|
break;
|
|
}
|
|
qPropToken = qPropToken->next();
|
|
}
|
|
}
|
|
}
|
|
|
|
if (settings->library.isreflection(tok->str())) {
|
|
const int index = settings->library.reflectionArgument(tok->str());
|
|
if (index >= 0) {
|
|
const Token * funcToken = tok->next();
|
|
int p = 0;
|
|
std::string value;
|
|
while (funcToken) {
|
|
if (funcToken->str()==",") {
|
|
if (++p==index)
|
|
break;
|
|
value = "";
|
|
} else
|
|
value += funcToken->str();
|
|
funcToken = funcToken->next();
|
|
}
|
|
if (p==index) {
|
|
value = value.substr(1, value.length() - 2);
|
|
_functions[value].usedOtherFile = true;
|
|
}
|
|
}
|
|
}
|
|
|
|
const Token *funcname = nullptr;
|
|
|
|
if (tok->scope()->isExecutable() && Token::Match(tok->next(), "%var% (")) {
|
|
funcname = tok->next();
|
|
}
|
|
|
|
else if (tok->scope()->isExecutable() && Token::Match(tok->next(), "%var% <") && Token::simpleMatch(tok->linkAt(2), "> (")) {
|
|
funcname = tok->next();
|
|
}
|
|
|
|
else if (Token::Match(tok, "[;{}.,()[=+-/|!?:] &| %var% [(),;]:}]")) {
|
|
funcname = tok->next();
|
|
if (funcname->str() == "&")
|
|
funcname = funcname->next();
|
|
}
|
|
|
|
else if (Token::Match(tok, "[;{}.,()[=+-/|!?:] &| %var% :: %var%")) {
|
|
funcname = tok->next();
|
|
if (funcname->str() == "&")
|
|
funcname = funcname->next();
|
|
while (Token::Match(funcname,"%var% :: %var%"))
|
|
funcname = funcname->tokAt(2);
|
|
if (!Token::Match(funcname, "%var% [(),;:}]"))
|
|
continue;
|
|
}
|
|
|
|
else
|
|
continue;
|
|
|
|
// funcname ( => Assert that the end parentheses isn't followed by {
|
|
if (Token::Match(funcname, "%var% (|<")) {
|
|
const Token *ftok = funcname->next();
|
|
if (ftok->str() == "<")
|
|
ftok = ftok->link();
|
|
if (Token::Match(ftok->linkAt(1), ") const|throw|{"))
|
|
funcname = nullptr;
|
|
}
|
|
|
|
if (funcname) {
|
|
FunctionUsage &func = _functions[ funcname->str()];
|
|
|
|
if (func.filename.empty() || func.filename == "+")
|
|
func.usedOtherFile = true;
|
|
else
|
|
func.usedSameFile = true;
|
|
}
|
|
}
|
|
}
|
|
|
|
|
|
|
|
|
|
void CheckUnusedFunctions::check(ErrorLogger * const errorLogger)
|
|
{
|
|
for (std::map<std::string, FunctionUsage>::const_iterator it = _functions.begin(); it != _functions.end(); ++it) {
|
|
const FunctionUsage &func = it->second;
|
|
if (func.usedOtherFile || func.filename.empty())
|
|
continue;
|
|
if (it->first == "main" ||
|
|
it->first == "WinMain" ||
|
|
it->first == "_tmain" ||
|
|
it->first == "if" ||
|
|
(it->first.compare(0, 8, "operator") == 0 && it->first.size() > 8 && !std::isalnum(it->first[8])))
|
|
continue;
|
|
if (! func.usedSameFile) {
|
|
std::string filename;
|
|
if (func.filename == "+")
|
|
filename = "";
|
|
else
|
|
filename = func.filename;
|
|
unusedFunctionError(errorLogger, filename, func.lineNumber, it->first);
|
|
} else if (! func.usedOtherFile) {
|
|
/** @todo add error message "function is only used in <file> it can be static" */
|
|
/*
|
|
std::ostringstream errmsg;
|
|
errmsg << "The function '" << it->first << "' is only used in the file it was declared in so it should have local linkage.";
|
|
_errorLogger->reportErr( errmsg.str() );
|
|
*/
|
|
}
|
|
}
|
|
}
|
|
|
|
void CheckUnusedFunctions::unusedFunctionError(ErrorLogger * const errorLogger,
|
|
const std::string &filename, unsigned int lineNumber,
|
|
const std::string &funcname)
|
|
{
|
|
std::list<ErrorLogger::ErrorMessage::FileLocation> locationList;
|
|
if (!filename.empty()) {
|
|
ErrorLogger::ErrorMessage::FileLocation fileLoc;
|
|
fileLoc.setfile(filename);
|
|
fileLoc.line = lineNumber;
|
|
locationList.push_back(fileLoc);
|
|
}
|
|
|
|
const ErrorLogger::ErrorMessage errmsg(locationList, Severity::style, "The function '" + funcname + "' is never used.", "unusedFunction", false);
|
|
if (errorLogger)
|
|
errorLogger->reportErr(errmsg);
|
|
else
|
|
reportError(errmsg);
|
|
}
|