walkero
/
cppcheck
1
0
Fork 0
Code Issues Pull Requests Projects Releases 1 Wiki Activity

Fixed #4035 (False positive: Memory leak: pTempFile)

This commit is contained in:
Daniel Marjamäki 2012-08-25 10:55:20 +02:00
parent 808c3468c9
commit 5e1ccfaf90
3 changed files with 19 additions and 20 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
lib/tokenize.cpp
View File

@ -1995,8 +1995,6 @@ bool Tokenizer::tokenize(std::istream &code,
removeRedundantSemicolons(); removeRedundantSemicolons();
simplifyReservedWordNullptr();
simplifyParameterVoid(); simplifyParameterVoid();
simplifyRedundantConsecutiveBraces(); simplifyRedundantConsecutiveBraces();
@ -2106,6 +2104,14 @@ void Tokenizer::simplifyNull()
tok->str("0"); tok->str("0");
} }
} }
// nullptr..
if (isCPP()) {
for (Token *tok = list.front(); tok; tok = tok->next()) {
if (tok->str() == "nullptr")
tok->str("0");
}
}
} }
void Tokenizer::concatenateNegativeNumber() void Tokenizer::concatenateNegativeNumber()
@ -2198,16 +2204,6 @@ void Tokenizer::simplifyParameterVoid()
} }
} }
void Tokenizer::simplifyReservedWordNullptr()
{
if (_settings->standards.cpp11) {
for (Token *tok = list.front(); tok; tok = tok->next()) {
if (tok->str() == "nullptr")
tok->str("0");
}
}
}
void Tokenizer::simplifyRedundantConsecutiveBraces() void Tokenizer::simplifyRedundantConsecutiveBraces()
{ {
// Remove redundant consecutive braces, i.e. '.. { { .. } } ..' -> '.. { .. } ..'. // Remove redundant consecutive braces, i.e. '.. { { .. } } ..' -> '.. { .. } ..'.

2
lib/tokenize.h
View File

@ -463,8 +463,6 @@ public:
bool hasComplicatedSyntaxErrorsInTemplates(); bool hasComplicatedSyntaxErrorsInTemplates();
void simplifyReservedWordNullptr();
/** /**
* Simplify e.g. 'atol("0")' into '0' * Simplify e.g. 'atol("0")' into '0'
*/ */

5
test/testtokenize.cpp
View File

@ -4646,6 +4646,7 @@ private:
} }
void simplify_null() { void simplify_null() {
{
const char code[] = const char code[] =
"int * p = NULL;\n" "int * p = NULL;\n"
"int * q = __null;\n"; "int * q = __null;\n";
@ -4654,6 +4655,10 @@ private:
ASSERT_EQUALS(expected, tokenizeAndStringify(code,true)); ASSERT_EQUALS(expected, tokenizeAndStringify(code,true));
} }
ASSERT_EQUALS("( a == nullptr )", tokenizeAndStringify("(a==nullptr)", false, false, Settings::Unspecified, "test.c"));
ASSERT_EQUALS("( ! a )", tokenizeAndStringify("(a==nullptr)", false, false, Settings::Unspecified, "test.cpp"));
}
void simplifyMulAndParens() { void simplifyMulAndParens() {
// (error) Resource leak // (error) Resource leak
const char code[] = "void f() {" const char code[] = "void f() {"