Skip to content

Commit fd3b00a

Browse files
committed
Simplify library define code
1 parent abdf5ff commit fd3b00a

2 files changed

Lines changed: 4 additions & 18 deletions

File tree

lib/cppcheck.cpp

Lines changed: 2 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -1049,16 +1049,8 @@ unsigned int CppCheck::checkInternal(const FileWithDetails& file, const std::str
10491049
std::set<std::string> configDefines = { "__cplusplus" };
10501050

10511051
// Insert library defines
1052-
for (const auto &define : mSettings.library.defines()) {
1053-
const std::string::size_type paren = define.find("(");
1054-
const std::string::size_type space = define.find(" ");
1055-
std::string::size_type end = space;
1056-
1057-
if (paren != std::string::npos && paren < space)
1058-
end = paren;
1059-
1060-
configDefines.insert(define.substr(0, end));
1061-
}
1052+
for (const auto &define : mSettings.library.defines())
1053+
configDefines.insert(define.substr(0, define.find_first_of("( ")));
10621054

10631055
preprocessor.setLoadCallback([&](simplecpp::FileData &data) {
10641056
// Do preprocessing on included file

test/testpreprocessor.cpp

Lines changed: 2 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -401,14 +401,8 @@ class TestPreprocessor : public TestFixture {
401401
Preprocessor preprocessor(tokens, settings, *this, Standards::Language::C);
402402
std::set<std::string> configs = { "" };
403403
std::set<std::string> configDefines = { "__cplusplus" };
404-
for (const auto &define : settings.library.defines()) {
405-
const std::string::size_type paren = define.find("(");
406-
const std::string::size_type space = define.find(" ");
407-
std::string::size_type end = space;
408-
if (paren != std::string::npos && paren < space)
409-
end = paren;
410-
configDefines.insert(define.substr(0, end));
411-
}
404+
for (const auto &define : settings.library.defines())
405+
configDefines.insert(define.substr(0, define.find_first_of("( ")));
412406
preprocessor.setLoadCallback([&](simplecpp::FileData &data) {
413407
Preprocessor::removeComments(data.tokens);
414408
preprocessor.getConfigs(data.filename, data.tokens, configDefines, configs);

0 commit comments

Comments
 (0)