fix ci

Author: dmcppcheck

cli/filelister.cpp

@@ -46,7 +46,7 @@
 // When compiling Unicode targets WinAPI automatically uses *W Unicode versions
 // of called functions. Thus, we explicitly call *A versions of the functions.
 
-static std::string addFiles2(std::list<FileWithDetails>&files, const std::string &path, const std::set<std::string> &extra, bool recursive, const PathMatch& ignored, bool debug = false)
+static std::string addFiles2(std::list<FileWithDetails>&files, const std::string &path, const std::set<std::string> &extra, bool recursive, PathMatch& ignored, bool debug = false)
 {
     const std::string cleanedPath = Path::toNativeSeparators(path);
 
@@ -163,7 +163,7 @@ static std::string addFiles2(std::list<FileWithDetails>&files, const std::string
     return "";
 }
 
-std::string FileLister::addFiles(std::list<FileWithDetails> &files, const std::string &path, const std::set<std::string> &extra, bool recursive, const PathMatch& ignored, bool debug)
+std::string FileLister::addFiles(std::list<FileWithDetails> &files, const std::string &path, const std::set<std::string> &extra, bool recursive, PathMatch& ignored, bool debug)
 {
     if (path.empty())
         return "no path specified";

lib/pathmatch.cpp

@@ -21,6 +21,7 @@
 #include "path.h"
 
 #include <algorithm>
+#include <iterator>
 #include <stack>
 #include <string>
 #include <utility>
@@ -155,9 +156,8 @@ bool PathMatch::match(const std::string &pattern, const std::string &path, const
 
 std::vector<std::string> PathMatch::unmatched() const {
     std::vector<std::string> ret;
-    for (const std::string& s: mPatterns) {
-        if (mMatchedPatterns.count(s) == 0)
-            ret.push_back(s);
-    }
+    std::copy_if(mPatterns.cbegin(), mPatterns.cend(), std::back_inserter(ret), [this](const std::string& s) {
+        return mMatchedPatterns.count(s) == 0;
+    });
     return ret;
 }

lib/pathmatch.h

@@ -24,6 +24,7 @@
 #include <cctype>
 #include <cstdint>
 #include <cstdio>
+#include <set>
 #include <string>
 #include <vector>