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

Merge branch 'master' of git@github.com:danmar/cppcheck

This commit is contained in:
Daniel Marjamäki 2010-03-28 15:56:38 +02:00
parent 62d2845014 417f14fccf
commit 9656379ffb
2 changed files with 43 additions and 18 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
lib/preprocessor.cpp
View File

@ -353,7 +353,8 @@ std::string Preprocessor::removeParantheses(const std::string &str)
} }
// "#if(A) => #if A", but avoid "#if (defined A) || defined (B)" // "#if(A) => #if A", but avoid "#if (defined A) || defined (B)"
if (line.compare(0, 4, "#if(") == 0 && line[line.length() - 1] == ')') if ((line.compare(0, 4, "#if(") == 0 || line.compare(0, 6, "#elif(") == 0) &&
line[line.length() - 1] == ')')
{ {
int ind = 0; int ind = 0;
for (std::string::size_type i = 0; i < line.length(); ++i) for (std::string::size_type i = 0; i < line.length(); ++i)
@ -367,7 +368,7 @@ std::string Preprocessor::removeParantheses(const std::string &str)
{ {
if (i == line.length() - 1) if (i == line.length() - 1)
{ {
line[3] = ' '; line[line.find('(')] = ' ';
line.erase(line.length() - 1); line.erase(line.length() - 1);
} }
break; break;
@ -378,7 +379,7 @@ std::string Preprocessor::removeParantheses(const std::string &str)
if (line.compare(0, 4, "#if(") == 0) if (line.compare(0, 4, "#if(") == 0)
line.insert(3, " "); line.insert(3, " ");
else if (line.compare(0, 4, "#elif(") == 0) else if (line.compare(0, 6, "#elif(") == 0)
line.insert(5, " "); line.insert(5, " ");
} }
ret << line << "\n"; ret << line << "\n";

24
test/testpreprocessor.cpp
View File

@ -615,6 +615,7 @@ private:
void elif() void elif()
{
{ {
const char filedata[] = "#if DEF1\n" const char filedata[] = "#if DEF1\n"
"ABC\n" "ABC\n"
@ -635,6 +636,29 @@ private:
ASSERT_EQUALS(3, static_cast<unsigned int>(actual.size())); ASSERT_EQUALS(3, static_cast<unsigned int>(actual.size()));
} }
{
const char filedata[] = "#if(defined DEF1)\n"
"ABC\n"
"#elif(defined DEF2)\n"
"DEF\n"
"#else\n"
"GHI\n"
"#endif\n";
// Preprocess => actual result..
std::istringstream istr(filedata);
std::map<std::string, std::string> actual;
Preprocessor preprocessor;
preprocessor.preprocess(istr, actual, "file.c");
// Compare results..
ASSERT_EQUALS("\n\n\n\n\nGHI\n\n", actual[""]);
ASSERT_EQUALS("\nABC\n\n\n\n\n\n", actual["DEF1"]);
ASSERT_EQUALS("\n\n\nDEF\n\n\n\n", actual["DEF2"]);
ASSERT_EQUALS(3, static_cast<unsigned int>(actual.size()));
}
}
void match_cfg_def() void match_cfg_def()