Filter out duplicate error messages in cli.

aggro80 · aggro80 · commit e87628b72fb6 · 2011-10-09T21:03:38.000+03:00
diff --git a/cli/cppcheckexecutor.cpp b/cli/cppcheckexecutor.cpp
@@ -255,6 +255,11 @@ int CppCheckExecutor::check(int argc, const char* const argv[])
 
 void CppCheckExecutor::reportErr(const std::string &errmsg)
 {
+    // Alert only about unique errors
+    if (std::find(_errorList.begin(), _errorList.end(), errmsg) != _errorList.end())
+        return;
+
+    _errorList.push_back(errmsg);
     std::cerr << errmsg << std::endl;
 }
 
diff --git a/cli/cppcheckexecutor.h b/cli/cppcheckexecutor.h
@@ -107,6 +107,11 @@ class CppCheckExecutor : public ErrorLogger
 
 private:
 
+    /**
+     * Used to filter out duplicate error messages.
+     */
+    std::list<std::string> _errorList;
+
     /**
      * Report progress time
      */

Original file line number	Diff line number	Diff line change
`@@ -255,6 +255,11 @@ int CppCheckExecutor::check(int argc, const char* const argv[])`
`255`	`255`
`256`	`256`	`void CppCheckExecutor::reportErr(const std::string &errmsg)`
`257`	`257`	`{`
	`258`	`+ // Alert only about unique errors`
	`259`	`+ if (std::find(_errorList.begin(), _errorList.end(), errmsg) != _errorList.end())`
	`260`	`+ return;`
	`261`	`+`
	`262`	`+ _errorList.push_back(errmsg);`
`258`	`263`	`std::cerr << errmsg << std::endl;`
`259`	`264`	`}`
`260`	`265`