From 83e3ccab63825ed351651c9be2b0be580eacc398 Mon Sep 17 00:00:00 2001 From: firewave Date: Fri, 19 Jan 2024 01:18:00 +0100 Subject: [PATCH] CppCheck: only invoke `CheckUnusedFunctions::check()` if `unusedFunctions` is actually enabled --- lib/cppcheck.cpp | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/lib/cppcheck.cpp b/lib/cppcheck.cpp index 861fd372989..f3dd59d9fd2 100644 --- a/lib/cppcheck.cpp +++ b/lib/cppcheck.cpp @@ -1793,7 +1793,8 @@ bool CppCheck::analyseWholeProgram() for (Check *check : Check::instances()) errors |= check->analyseWholeProgram(&ctu, mFileInfo, mSettings, *this); // TODO: ctu - errors |= CheckUnusedFunctions::check(mSettings, *this); + if (mSettings.checks.isEnabled(Checks::unusedFunction)) + errors |= CheckUnusedFunctions::check(mSettings, *this); return errors && (mExitCode > 0); } @@ -1859,7 +1860,8 @@ void CppCheck::analyseWholeProgram(const std::string &buildDir, const std::list< for (Check *check : Check::instances()) check->analyseWholeProgram(&ctuFileInfo, fileInfoList, mSettings, *this); - CheckUnusedFunctions::check(mSettings, *this); + if (mSettings.checks.isEnabled(Checks::unusedFunction)) + CheckUnusedFunctions::check(mSettings, *this); for (Check::FileInfo *fi : fileInfoList) delete fi;