Skip to content

Commit 028e360

Browse files
committed
Add some comments
1 parent c96c859 commit 028e360

1 file changed

Lines changed: 4 additions & 0 deletions

File tree

lib/cppcheck.cpp

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1036,15 +1036,19 @@ unsigned int CppCheck::checkInternal(const FileWithDetails& file, const std::str
10361036
preprocessor.createDirectives(tokens1, directives);
10371037
preprocessor.simplifyPragmaAsm(tokens1);
10381038

1039+
// This needs to be a linked list to allow new configurations to be discovered
1040+
// and added while iterating and checking existing configurations
10391041
std::list<std::string> configurations;
10401042
std::set<std::string> configDefines = { "__cplusplus" };
10411043

10421044
preprocessor.setLoadCallback([&](simplecpp::FileData& data) {
1045+
// Do preprocessing on included file
10431046
preprocessor.addRemarkComments(data.tokens, mLogger->remarkComments());
10441047
preprocessor.inlineSuppressions(data.tokens, mSuppressions.nomsg);
10451048
data.tokens.removeComments();
10461049
preprocessor.createDirectives(data.tokens, directives);
10471050
preprocessor.simplifyPragmaAsm(data.tokens);
1051+
// Discover new configurations from included file
10481052
if ((mSettings.checkAllConfigurations && mSettings.userDefines.empty()) || mSettings.force)
10491053
preprocessor.getConfigs(data.filename, data.tokens, configDefines, configurations);
10501054
});

0 commit comments

Comments
 (0)