do not execute addons with UNUSEDFUNCTION_ONLY hack

Author: firewave

lib/cppcheck.cpp

@@ -1495,7 +1495,7 @@ void CppCheck::executeAddons(const std::string& dumpFile, const FileWithDetails&
 
 void CppCheck::executeAddons(const std::vector<std::string>& files, const std::string& file0)
 {
-    if (mSettings.addons.empty() || files.empty())
+    if (mSettings.addons.empty() || files.empty() || Settings::unusedFunctionOnly())
         return;
 
     const bool isCtuInfo = endsWith(files[0], ".ctu-info");

test/cli/other_test.py

@@ -4104,7 +4104,6 @@ def test_active_unusedfunction_only(tmp_path):
 
 
 def test_active_unusedfunction_only_builddir(tmp_path):
-    # TODO: should only report a single active check
     checkers_exp = [
         'CheckUnusedFunctions::check'
     ]
@@ -4113,143 +4112,13 @@ def test_active_unusedfunction_only_builddir(tmp_path):
 
 def test_active_unusedfunction_only_misra(tmp_path):
     # TODO: should only report a single active check
-    __test_active_checkers(tmp_path, 263, 1166, use_unusedfunction_only=True, use_misra=True)
+    __test_active_checkers(tmp_path, 7, 1166, use_unusedfunction_only=True, use_misra=True)
 
 
 @pytest.mark.xfail(strict=True)  # TODO: active count and checkers.txt differ
 def test_active_unusedfunction_only_misra_builddir(tmp_path):
     # TODO: should only report a single active check
     checkers_exp = [
-        'CheckUnusedFunctions::check',
-        'Misra C: 1.2',
-        'Misra C: 1.4',
-        'Misra C: 10.1',
-        'Misra C: 10.2',
-        'Misra C: 10.3',
-        'Misra C: 10.4',
-        'Misra C: 10.5',
-        'Misra C: 10.6',
-        'Misra C: 10.7',
-        'Misra C: 10.8',
-        'Misra C: 11.1',
-        'Misra C: 11.2',
-        'Misra C: 11.3',
-        'Misra C: 11.4',
-        'Misra C: 11.5',
-        'Misra C: 11.6',
-        'Misra C: 11.7',
-        'Misra C: 11.8',
-        'Misra C: 11.9',
-        'Misra C: 12.1',
-        'Misra C: 12.2',
-        'Misra C: 12.3',
-        'Misra C: 12.4',
-        'Misra C: 13.1',
-        'Misra C: 13.3',
-        'Misra C: 13.4',
-        'Misra C: 13.5',
-        'Misra C: 13.6',
-        'Misra C: 14.1',
-        'Misra C: 14.2',
-        'Misra C: 14.4',
-        'Misra C: 15.1',
-        'Misra C: 15.2',
-        'Misra C: 15.3',
-        'Misra C: 15.4',
-        'Misra C: 15.5',
-        'Misra C: 15.6',
-        'Misra C: 15.7',
-        'Misra C: 16.1',
-        'Misra C: 16.2',
-        'Misra C: 16.3',
-        'Misra C: 16.4',
-        'Misra C: 16.5',
-        'Misra C: 16.6',
-        'Misra C: 16.7',
-        'Misra C: 17.1',
-        'Misra C: 17.2',
-        'Misra C: 17.3',
-        'Misra C: 17.6',
-        'Misra C: 17.7',
-        'Misra C: 17.8',
-        'Misra C: 18.4',
-        'Misra C: 18.5',
-        'Misra C: 18.7',
-        'Misra C: 18.8',
-        'Misra C: 19.2',
-        'Misra C: 2.2',
-        'Misra C: 2.3',
-        'Misra C: 2.4',
-        'Misra C: 2.5',
-        'Misra C: 2.7',
-        'Misra C: 20.1',
-        'Misra C: 20.10',
-        'Misra C: 20.11',
-        'Misra C: 20.12',
-        'Misra C: 20.13',
-        'Misra C: 20.14',
-        'Misra C: 20.2',
-        'Misra C: 20.3',
-        'Misra C: 20.4',
-        'Misra C: 20.5',
-        'Misra C: 20.7',
-        'Misra C: 20.8',
-        'Misra C: 20.9',
-        'Misra C: 21.1',
-        'Misra C: 21.10',
-        'Misra C: 21.11',
-        'Misra C: 21.12',
-        'Misra C: 21.14',
-        'Misra C: 21.15',
-        'Misra C: 21.16',
-        'Misra C: 21.19',
-        'Misra C: 21.2',
-        'Misra C: 21.20',
-        'Misra C: 21.21',
-        'Misra C: 21.3',
-        'Misra C: 21.4',
-        'Misra C: 21.5',
-        'Misra C: 21.6',
-        'Misra C: 21.7',
-        'Misra C: 21.8',
-        'Misra C: 21.9',
-        'Misra C: 22.10',
-        'Misra C: 22.5',
-        'Misra C: 22.7',
-        'Misra C: 22.8',
-        'Misra C: 22.9',
-        'Misra C: 3.1',
-        'Misra C: 4.1',
-        'Misra C: 4.2',
-        'Misra C: 5.1',
-        'Misra C: 5.2',
-        'Misra C: 5.4',
-        'Misra C: 5.5',
-        'Misra C: 5.6',
-        'Misra C: 5.7',
-        'Misra C: 5.8',
-        'Misra C: 5.9',
-        'Misra C: 6.1',
-        'Misra C: 6.2',
-        'Misra C: 7.1',
-        'Misra C: 7.2',
-        'Misra C: 7.3',
-        'Misra C: 7.4',
-        'Misra C: 8.1',
-        'Misra C: 8.10',
-        'Misra C: 8.11',
-        'Misra C: 8.12',
-        'Misra C: 8.14',
-        'Misra C: 8.2',
-        'Misra C: 8.4',
-        'Misra C: 8.5',
-        'Misra C: 8.6',
-        'Misra C: 8.7',
-        'Misra C: 8.8',
-        'Misra C: 8.9',
-        'Misra C: 9.2',
-        'Misra C: 9.3',
-        'Misra C: 9.4',
-        'Misra C: 9.5'
-    ]
-    __test_active_checkers(tmp_path, 263, 1166, use_unusedfunction_only=True, use_misra=True, checkers_exp=checkers_exp)
+        'CheckUnusedFunctions::check'
+    ]
+    __test_active_checkers(tmp_path, 7, 1166, use_unusedfunction_only=True, use_misra=True, checkers_exp=checkers_exp)