Skip to content

Commit 49fed2d

Browse files
committed
refactor
1 parent 41b6f93 commit 49fed2d

1 file changed

Lines changed: 6 additions & 7 deletions

File tree

lib/cppcheck.cpp

Lines changed: 6 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1063,12 +1063,6 @@ unsigned int CppCheck::checkInternal(const FileWithDetails& file, const std::str
10631063
}
10641064
#endif
10651065

1066-
if (!mSettings.force &&
1067-
configurations.size() > mSettings.maxConfigs &&
1068-
mSettings.maxConfigs == 12 && // <- do not warn if maxConfigs has been changed
1069-
mSettings.severity.isEnabled(Severity::information))
1070-
tooManyConfigsError(Path::toNativeSeparators(file.spath()), configurations.size());
1071-
10721066
FilesDeleter filesDeleter;
10731067

10741068
// write dump file xml prolog
@@ -1093,8 +1087,13 @@ unsigned int CppCheck::checkInternal(const FileWithDetails& file, const std::str
10931087

10941088
// Check only a few configurations (default 12), after that bail out, unless --force
10951089
// was used.
1096-
if (!mSettings.force && ++checkCount > mSettings.maxConfigs)
1090+
if (!mSettings.force && ++checkCount > mSettings.maxConfigs) {
1091+
// if maxConfigs has default value then report information message that configurations are skipped
1092+
if (mSettings.maxConfigs == 12 && mSettings.severity.isEnabled(Severity::information))
1093+
tooManyConfigsError(Path::toNativeSeparators(file.spath()), configurations.size());
1094+
10971095
break;
1096+
}
10981097

10991098
std::string currentConfig;
11001099

0 commit comments

Comments
 (0)