cfcfa3f000
Settings-class currently enables style checking via dedicated boolean attribute. All other CLI's enable-options are handled through the enable-list. This commit moves style-check enabling to use the enable-list. Main advantage is the consistency how options are handled/stored in the Settings class. Which also unifies using them for the other code. You need to enable certain type of checks? Use the addEnabled()-method. You want to check if certain type of checks are enabled? Use the isEnabled()-method.
60 lines
2.2 KiB
C++
60 lines
2.2 KiB
C++
/*
|
|
* Cppcheck - A tool for static C/C++ code analysis
|
|
* Copyright (C) 2007-2011 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/>.
|
|
*/
|
|
|
|
//---------------------------------------------------------------------------
|
|
// Find non reentrant functions
|
|
//---------------------------------------------------------------------------
|
|
|
|
#include "checknonreentrantfunctions.h"
|
|
|
|
//---------------------------------------------------------------------------
|
|
|
|
|
|
// Register this check class (by creating a static instance of it)
|
|
namespace
|
|
{
|
|
CheckNonReentrantFunctions instance;
|
|
}
|
|
|
|
void CheckNonReentrantFunctions::nonReentrantFunctions()
|
|
{
|
|
if (!_settings->isEnabled("posix") || !_settings->isEnabled("style"))
|
|
return;
|
|
|
|
// Don't check C# and Java code
|
|
if (_tokenizer->isJavaOrCSharp())
|
|
return;
|
|
|
|
for (const Token *tok = _tokenizer->tokens(); tok; tok = tok->next())
|
|
{
|
|
std::list< std::pair<const std::string, const std::string> >::const_iterator it(_nonReentrantFunctions.begin()), itend(_nonReentrantFunctions.end());
|
|
for (; it!=itend; ++it)
|
|
{
|
|
if (tok->strAt(1) == it->first && tok->strAt(2) == "(" && tok->tokAt(1)->varId() == 0 && !tok->tokAt(0)->isName() && !Token::Match(tok, ".|::|:|,"))
|
|
{
|
|
// If checking code that is single threaded, this might be not interesing for all.
|
|
// Therefore this is "portabiblity"
|
|
reportError(tok->tokAt(1), Severity::portability, "nonreentrantFunctions"+it->first, it->second);
|
|
break;
|
|
}
|
|
}
|
|
}
|
|
}
|
|
//---------------------------------------------------------------------------
|
|
|