ci fixes

Author: ludviggunne

lib/errorlogger.h

@@ -326,7 +326,7 @@ class CPPCHECKLIB ErrorLogger {
 protected:
     virtual std::size_t getSourceCacheSize() const {
         return 4;
-    };
+    }
 
     class SourceCacheEntry {
     public:

test/testerrorlogger.cpp

@@ -872,18 +872,18 @@ class TestErrorLogger : public TestFixture {
     public:
         friend class TestErrorLogger;
 
-        virtual void reportOut(const std::string &outmsg, Color c)
+        virtual void reportOut(const std::string &outmsg, Color c) override
         {
             (void) outmsg;
             (void) c;
         }
 
-        virtual void reportErr(const ErrorMessage &msg)
+        virtual void reportErr(const ErrorMessage &msg) override
         {
             (void) msg.toString(false, "{code}", "{code}", getSourceLineCallback());
         }
 
-        virtual void reportMetric(const std::string &metric)
+        virtual void reportMetric(const std::string &metric) override
         {
             (void) metric;
         }
@@ -920,8 +920,8 @@ class TestErrorLogger : public TestFixture {
         };
 
         std::list<ErrorMessage::FileLocation> callstack;
-        for (const auto &file : callstackFiles)
-            callstack.emplace_back(file, 1, 1);
+        for (const auto &filename : callstackFiles)
+            callstack.emplace_back(filename, 1, 1);
 
         const ErrorMessage msg(callstack,
                                file,
@@ -935,10 +935,10 @@ class TestErrorLogger : public TestFixture {
         ASSERT_EQUALS(content.size(), copy.size());
 
         std::make_heap(copy.begin(), copy.end(), heapCompare);
-        for (const auto match : content) {
+        for (const auto &match : content) {
             std::pop_heap(copy.begin(), copy.end(), heapCompare);
-            ASSERT_EQUALS(match.file, copy.back()->file);
-            ASSERT_EQUALS(match.prio, copy.back()->prio);
+            ASSERT_EQUALS_LOC(match.file, copy.back()->file, testfile, testline);
+            ASSERT_EQUALS_LOC(match.prio, copy.back()->prio, testfile, testline);
             copy.pop_back();
         }
     }