suprr

Author: firewave

gui/mainwindow.cpp

@@ -1021,9 +1021,7 @@ QPair<bool,Settings> MainWindow::getCppcheckSettings()
             tryLoadLibrary(&result.library, filename);
         }
 
-        for (const Suppressions::Suppression &suppression : mProjectFile->getSuppressions()) {
-            result.nomsg.addSuppression(suppression);
-        }
+        result.nomsg.addSuppressions(mProjectFile->getSuppressions().toStdList()); // TODO: check result
 
         // Only check the given -D configuration
         if (!defines.isEmpty())

lib/cppcheck.cpp

@@ -581,7 +581,11 @@ unsigned int CppCheck::check(const FileSettings &fs)
         return temp.check(Path::simplifyPath(fs.filename));
     }
     const unsigned int returnValue = temp.checkFile(Path::simplifyPath(fs.filename), fs.cfg);
-    mSettings.nomsg.addSuppressions(temp.mSettings.nomsg.getSuppressions());
+    for (const auto& suppr : temp.mSettings.nomsg.getSuppressions())
+    {
+        const bool res = mSettings.nomsg.updateSuppressionState(suppr);
+        assert(res);
+    }
     return returnValue;
 }
 

lib/preprocessor.cpp

@@ -254,7 +254,7 @@ static void addInlineSuppressions(const simplecpp::TokenList &tokens, const Sett
                             suppr.lineNumber = supprBegin->lineNumber;
                             suppr.type = Suppressions::Type::block;
                             inlineSuppressionsBlockBegin.erase(supprBegin);
-                            suppressions.addSuppression(std::move(suppr));
+                            suppressions.addSuppression(std::move(suppr)); // TODO: check result
                             throwError = false;
                             break;
                         }
@@ -279,10 +279,10 @@ static void addInlineSuppressions(const simplecpp::TokenList &tokens, const Sett
                 suppr.thisAndNextLine = thisAndNextLine;
                 suppr.lineNumber = tok->location.line;
                 suppr.macroName = macroName;
-                suppressions.addSuppression(std::move(suppr));
+                suppressions.addSuppression(std::move(suppr)); // TODO: check result
             } else if (Suppressions::Type::file == suppr.type) {
                 if (onlyComments)
-                    suppressions.addSuppression(std::move(suppr));
+                    suppressions.addSuppression(std::move(suppr)); // TODO: check result
                 else
                     bad.emplace_back(suppr.fileName, suppr.lineNumber, "File suppression should be at the top of the file");
             }

lib/suppressions.cpp

@@ -249,11 +249,7 @@ std::string Suppressions::addSuppression(Suppressions::Suppression suppression)
     auto foundSuppression = std::find_if(mSuppressions.begin(), mSuppressions.end(),
                                          std::bind(&Suppression::isSameParameters, &suppression, std::placeholders::_1));
     if (foundSuppression != mSuppressions.end()) {
-        if (suppression.checked)
-            foundSuppression->checked = suppression.checked;
-        if (suppression.matched)
-            foundSuppression->matched = suppression.matched;
-        return "";
+        return "suppression already exists";
     }
 
     // Check that errorId is valid..
@@ -289,6 +285,22 @@ std::string Suppressions::addSuppressions(std::list<Suppression> suppressions)
     return "";
 }
 
+bool Suppressions::updateSuppressionState(const Suppressions::Suppression& suppression)
+{
+    // Check if suppression is already in list
+    auto foundSuppression = std::find_if(mSuppressions.begin(), mSuppressions.end(),
+                                         std::bind(&Suppression::isSameParameters, &suppression, std::placeholders::_1));
+    if (foundSuppression != mSuppressions.end()) {
+        if (suppression.checked)
+            foundSuppression->checked = suppression.checked;
+        if (suppression.matched)
+            foundSuppression->matched = suppression.matched;
+        return true;
+    }
+
+    return false;
+}
+
 void Suppressions::ErrorMessage::setFileName(std::string s)
 {
     mFileName = Path::simplifyPath(std::move(s));

lib/suppressions.h

@@ -194,6 +194,13 @@ class CPPCHECKLIB Suppressions {
      */
     std::string addSuppressions(std::list<Suppression> suppressions);
 
+    /**
+     * @brief Updates the state of the given suppression.
+     * @param suppression the suppression to update
+     * @return true if suppression to update was found
+     */
+    bool updateSuppressionState(const Suppression& suppression);
+
     /**
      * @brief Returns true if this message should not be shown to the user.
      * @param errmsg error message

test/testsuppressions.cpp

@@ -1467,6 +1467,8 @@ class TestSuppressions : public TestFixture {
         ASSERT_EQUALS(true, Suppressions::reportUnmatchedSuppressions(suppressions, *this));
         ASSERT_EQUALS("[a.c:10]: (information) Unmatched suppression: abc\n", errout.str());
     }
+
+    // TODO: test updateSuppressionState
 };
 
 REGISTER_TEST(TestSuppressions)