Rename errorlist=>mShowAllErrors

danmar · danmar · commit 96a1c6dec595 · 2019-07-15T18:31:12.000+02:00
diff --git a/cli/cppcheckexecutor.cpp b/cli/cppcheckexecutor.cpp
@@ -79,7 +79,7 @@
 /*static*/ FILE* CppCheckExecutor::mExceptionOutput = stdout;
 
 CppCheckExecutor::CppCheckExecutor()
-    : mSettings(nullptr), latestProgressOutputTime(0), mErrorOutput(nullptr), errorlist(false)
+    : mSettings(nullptr), latestProgressOutputTime(0), mErrorOutput(nullptr), mShowAllErrors(false)
 {
 }
 
@@ -105,7 +105,7 @@ bool CppCheckExecutor::parseFromArgs(CppCheck *cppcheck, int argc, const char* c
         }
 
         if (parser.getShowErrorMessages()) {
-            errorlist = true;
+            mShowAllErrors = true;
             std::cout << ErrorLogger::ErrorMessage::getXMLHeader();
             cppcheck->getErrorMessages();
             std::cout << ErrorLogger::ErrorMessage::getXMLFooter() << std::endl;
@@ -1054,7 +1054,7 @@ void CppCheckExecutor::reportStatus(std::size_t fileindex, std::size_t filecount
 
 void CppCheckExecutor::reportErr(const ErrorLogger::ErrorMessage &msg)
 {
-    if (errorlist) {
+    if (mShowAllErrors) {
         reportOut(msg.toXML());
     } else if (mSettings->xml) {
         reportErr(msg.toXML());
diff --git a/cli/cppcheckexecutor.h b/cli/cppcheckexecutor.h
@@ -190,7 +190,7 @@ class CppCheckExecutor : public ErrorLogger {
     /**
      * Has --errorlist been given?
      */
-    bool errorlist;
+    bool mShowAllErrors;
 };
 
 #endif // CPPCHECKEXECUTOR_H

Original file line number	Diff line number	Diff line change
`@@ -79,7 +79,7 @@`
`79`	`79`	`/static/ FILE* CppCheckExecutor::mExceptionOutput = stdout;`
`80`	`80`
`81`	`81`	`CppCheckExecutor::CppCheckExecutor()`
`82`		`- : mSettings(nullptr), latestProgressOutputTime(0), mErrorOutput(nullptr), errorlist(false)`
	`82`	`+ : mSettings(nullptr), latestProgressOutputTime(0), mErrorOutput(nullptr), mShowAllErrors(false)`
`83`	`83`	`{`
`84`	`84`	`}`
`85`	`85`
`@@ -105,7 +105,7 @@ bool CppCheckExecutor::parseFromArgs(CppCheck cppcheck, int argc, const char c`
`105`	`105`	`}`
`106`	`106`
`107`	`107`	`if (parser.getShowErrorMessages()) {`
`108`		`- errorlist = true;`
	`108`	`+ mShowAllErrors = true;`
`109`	`109`	`std::cout << ErrorLogger::ErrorMessage::getXMLHeader();`
`110`	`110`	`cppcheck->getErrorMessages();`
`111`	`111`	`std::cout << ErrorLogger::ErrorMessage::getXMLFooter() << std::endl;`
`@@ -1054,7 +1054,7 @@ void CppCheckExecutor::reportStatus(std::size_t fileindex, std::size_t filecount`
`1054`	`1054`
`1055`	`1055`	`void CppCheckExecutor::reportErr(const ErrorLogger::ErrorMessage &msg)`
`1056`	`1056`	`{`
`1057`		`- if (errorlist) {`
	`1057`	`+ if (mShowAllErrors) {`
`1058`	`1058`	`reportOut(msg.toXML());`
`1059`	`1059`	`} else if (mSettings->xml) {`
`1060`	`1060`	`reportErr(msg.toXML());`