summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorNoel Grandin <noel.grandin@collabora.co.uk>2024-11-29 15:06:48 +0200
committerNoel Grandin <noel.grandin@collabora.co.uk>2024-11-29 18:42:08 +0100
commit27509505be691b2f37d50d5c60ef1cea87357671 (patch)
tree9b75296d17a8d982e82f79dfdc3d75b3e0d8e9e7
parentf4fe6df6aa92573368c3fa0edb9fd03e64d9d059 (diff)
loplugin:unusedmethods tweak for less noise
Change-Id: Idedd1680b03f6b62ff80ad7d0fd74c11d1675053 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/177533 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
-rw-r--r--compilerplugins/clang/unusedmethods.cxx6
1 files changed, 6 insertions, 0 deletions
diff --git a/compilerplugins/clang/unusedmethods.cxx b/compilerplugins/clang/unusedmethods.cxx
index 5b10eae7e9df..db001f17620c 100644
--- a/compilerplugins/clang/unusedmethods.cxx
+++ b/compilerplugins/clang/unusedmethods.cxx
@@ -285,6 +285,8 @@ gotfunc:
logCallToRootMethods(calleeFunctionDecl, usedReturnSet);
return true;
}
+ if (auto attributedStmt = dyn_cast<AttributedStmt>(parent))
+ parent = attributedStmt->getSubStmt();
if (isa<Expr>(parent) || isa<ReturnStmt>(parent) || isa<DeclStmt>(parent)
|| isa<IfStmt>(parent) || isa<SwitchStmt>(parent) || isa<ForStmt>(parent)
|| isa<WhileStmt>(parent) || isa<DoStmt>(parent)
@@ -298,6 +300,10 @@ gotfunc:
{
return true;
}
+ report(
+ DiagnosticsEngine::Warning,
+ "unknown parent?",
+ parent->getBeginLoc());
parent->dump();
return true;
}