Skip to content

Commit

Permalink
checkersreport.cpp: fixed typo
Browse files Browse the repository at this point in the history
Co-authored-by: dcb314 <[email protected]>
  • Loading branch information
firewave and dcb314 committed Sep 19, 2024
1 parent 757620f commit 464a32f
Showing 1 changed file with 8 additions and 4 deletions.
12 changes: 8 additions & 4 deletions lib/checkersreport.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -141,11 +141,15 @@ std::string CheckersReport::getReport(const std::string& criticalErrors) const
fout << "Critical errors" << std::endl;
fout << "---------------" << std::endl;
if (!criticalErrors.empty()) {
fout << "There was critical errors (" << criticalErrors << ")" << std::endl;
fout << "All checking is skipped for a file with such error" << std::endl;
fout << "There were critical errors (" << criticalErrors << ")." << std::endl;
fout << "These cause the analysis of the file to end prematurely." << std::endl;
} else {
fout << "No critical errors, all files were checked." << std::endl;
fout << "Important: Analysis is still not guaranteed to be 'complete' it is possible there are false negatives." << std::endl;
fout << "No critical errors encountered." << std::endl;
// TODO: mention "information" and "debug" as source for indications of bailouts
// TODO: still rephrase this - this message does not provides confidence in the results
// TODO: document what a bailout is and why it is done - mention it in the upcoming security/tuning guide
// TODO: make bailouts a seperate group - need to differentiate between user bailouts (missing data like configuration/includes) and internal bailouts (e.g. limitations of ValueFlow)
fout << "Note: There might still have been non-critical bailouts which might lead to false negatives." << std::endl;
}

fout << std::endl << std::endl;
Expand Down

0 comments on commit 464a32f

Please sign in to comment.