[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[lmi-commits] [lmi] master d90e9fb 01/12: Ensure that test_coding_rules
From: |
Greg Chicares |
Subject: |
[lmi-commits] [lmi] master d90e9fb 01/12: Ensure that test_coding_rules exits with failure on any error |
Date: |
Mon, 28 Jun 2021 18:44:27 -0400 (EDT) |
branch: master
commit d90e9fb00109220767532fa42a31ee3d3ffb3efb
Author: Vadim Zeitlin <vadim@tt-solutions.com>
Commit: Gregory W. Chicares <gchicares@sbcglobal.net>
Ensure that test_coding_rules exits with failure on any error
Don't exit with success exit status if any errors were reported.
---
test_coding_rules.cpp | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/test_coding_rules.cpp b/test_coding_rules.cpp
index 690099f..6aa5e39 100644
--- a/test_coding_rules.cpp
+++ b/test_coding_rules.cpp
@@ -259,8 +259,11 @@ bool file::phyloanalyze(std::string const& s) const
return boost::regex_search(file_name(), boost::regex(s));
}
+bool error_flag = false;
+
void complain(file const& f, std::string const& complaint)
{
+ error_flag = true;
std::cout << "File '" << f.full_name() << "' " << complaint << std::endl;
}
@@ -1294,7 +1297,6 @@ statistics process_file(std::string const& file_path)
int try_main(int argc, char* argv[])
{
- bool error_flag = false;
statistics z;
for(int j = 1; j < argc; ++j)
{
- [lmi-commits] [lmi] master updated (016bf00 -> 04ec593), Greg Chicares, 2021/06/28
- [lmi-commits] [lmi] master d90e9fb 01/12: Ensure that test_coding_rules exits with failure on any error,
Greg Chicares <=
- [lmi-commits] [lmi] master 8882ef7 07/12: Use contains() for the lmi URL check, Greg Chicares, 2021/06/28
- [lmi-commits] [lmi] master 07edbd8 10/12: Document the One True nonempty d-char-sequence, Greg Chicares, 2021/06/28
- [lmi-commits] [lmi] master 4bcdbdb 11/12: Make a test more stringent, Greg Chicares, 2021/06/28
- [lmi-commits] [lmi] master 30ba88a 02/12: Remove check for Latin-9 from coding rules test, Greg Chicares, 2021/06/28
- [lmi-commits] [lmi] master 0f1bc07 03/12: Split the check for canonical header guards in two parts, Greg Chicares, 2021/06/28
- [lmi-commits] [lmi] master c1edfef 04/12: Use more standard regex syntax in the raw C++11 strings check, Greg Chicares, 2021/06/28
- [lmi-commits] [lmi] master 81c56f8 05/12: Add a test for detecting mis-sorted #include directives, Greg Chicares, 2021/06/28
- [lmi-commits] [lmi] master 72a9f51 06/12: Add a test for for-range declaration coding style checks, Greg Chicares, 2021/06/28
- [lmi-commits] [lmi] master b719340 09/12: Reword, Greg Chicares, 2021/06/28
- [lmi-commits] [lmi] master a9bc62f 08/12: Count coding-rules complaints, Greg Chicares, 2021/06/28