avoid redundant `simplecpp::Output::type` switch blocks (#5005)

* avoid redundant `simplecpp::Output::type` switch blocks

* fixed `useStlAlgorithm` warnings

lib/cppcheck.cpp

@@ -666,24 +666,11 @@ unsigned int CppCheck::checkFile(const std::string& filename, const std::string
         simplecpp::TokenList tokens1(fileStream, files, filename, &outputList);
 
         // If there is a syntax error, report it and stop
-        for (const simplecpp::Output &output : outputList) {
+        const auto output_it = std::find_if(outputList.cbegin(), outputList.cend(), [](const simplecpp::Output &output){
-            bool err;
+            return Preprocessor::hasErrors(output);
-            switch (output.type) {
+        });
-            case simplecpp::Output::ERROR:
+        if (output_it != outputList.cend()) {
-            case simplecpp::Output::INCLUDE_NESTED_TOO_DEEPLY:
+            const simplecpp::Output &output = *output_it;
-            case simplecpp::Output::SYNTAX_ERROR:
-            case simplecpp::Output::UNHANDLED_CHAR_ERROR:
-            case simplecpp::Output::EXPLICIT_INCLUDE_NOT_FOUND:
-                err = true;
-                break;
-            case simplecpp::Output::WARNING:
-            case simplecpp::Output::MISSING_HEADER:
-            case simplecpp::Output::PORTABILITY_BACKSLASH:
-                err = false;
-                break;
-            }
-
-            if (err) {
             std::string file = Path::fromNativeSeparators(output.location.file());
             if (mSettings.relativePaths)
                 file = Path::getRelativePath(file, mSettings.basePaths);
@@ -700,7 +687,6 @@ unsigned int CppCheck::checkFile(const std::string& filename, const std::string
             reportErr(errmsg);
             return mExitCode;
         }
-        }
 
         if (!preprocessor.loadFiles(tokens1, files))
             return mExitCode;
@@ -797,10 +783,10 @@ unsigned int CppCheck::checkFile(const std::string& filename, const std::string
         }
 
         // Run define rules on raw code
-        const auto it = std::find_if(mSettings.rules.cbegin(), mSettings.rules.cend(), [](const Settings::Rule& rule) {
+        const auto rules_it = std::find_if(mSettings.rules.cbegin(), mSettings.rules.cend(), [](const Settings::Rule& rule) {
             return rule.tokenlist == "define";
         });
-        if (it != mSettings.rules.cend()) {
+        if (rules_it != mSettings.rules.cend()) {
             std::string code;
             const std::list<Directive> &directives = preprocessor.getDirectives();
             for (const Directive &dir : directives) {

lib/preprocessor.cpp

@@ -627,10 +627,9 @@ static simplecpp::DUI createDUI(const Settings &mSettings, const std::string &cf
     return dui;
 }
 
-bool Preprocessor::hasErrors(const simplecpp::OutputList &outputList)
+bool Preprocessor::hasErrors(const simplecpp::Output &output)
 {
-    for (simplecpp::OutputList::const_iterator it = outputList.cbegin(); it != outputList.cend(); ++it) {
+    switch (output.type) {
-        switch (it->type) {
     case simplecpp::Output::ERROR:
     case simplecpp::Output::INCLUDE_NESTED_TOO_DEEPLY:
     case simplecpp::Output::SYNTAX_ERROR:
@@ -642,28 +641,27 @@ bool Preprocessor::hasErrors(const simplecpp::OutputList &outputList)
     case simplecpp::Output::PORTABILITY_BACKSLASH:
         break;
     }
-    }
     return false;
 }
 
+bool Preprocessor::hasErrors(const simplecpp::OutputList &outputList)
+{
+    const auto it = std::find_if(outputList.cbegin(), outputList.cend(), [](const simplecpp::Output &output) {
+        return hasErrors(output);
+    });
+    return it != outputList.cend();
+}
+
 void Preprocessor::handleErrors(const simplecpp::OutputList& outputList, bool throwError)
 {
     const bool showerror = (!mSettings.userDefines.empty() && !mSettings.force);
     reportOutput(outputList, showerror);
     if (throwError) {
-        for (const simplecpp::Output& output : outputList) {
+        const auto it = std::find_if(outputList.cbegin(), outputList.cend(), [](const simplecpp::Output &output){
-            switch (output.type) {
+            return hasErrors(output);
-            case simplecpp::Output::ERROR:
+        });
-            case simplecpp::Output::INCLUDE_NESTED_TOO_DEEPLY:
+        if (it != outputList.cend()) {
-            case simplecpp::Output::SYNTAX_ERROR:
+            throw *it;
-            case simplecpp::Output::UNHANDLED_CHAR_ERROR:
-            case simplecpp::Output::EXPLICIT_INCLUDE_NOT_FOUND:
-                throw output;
-            case simplecpp::Output::WARNING:
-            case simplecpp::Output::MISSING_HEADER:
-            case simplecpp::Output::PORTABILITY_BACKSLASH:
-                break;
-            }
         }
     }
 }

lib/preprocessor.h

@@ -180,6 +180,8 @@ public:
 
     void reportOutput(const simplecpp::OutputList &outputList, bool showerror);
 
+    static bool hasErrors(const simplecpp::Output &output);
+
 private:
     void missingInclude(const std::string &filename, unsigned int linenr, const std::string &header, HeaderTypes headerType);
     void error(const std::string &filename, unsigned int linenr, const std::string &msg);