diff options
author | Noel Grandin <noel.grandin@collabora.co.uk> | 2024-11-29 15:06:48 +0200 |
---|---|---|
committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2024-11-29 18:42:08 +0100 |
commit | 27509505be691b2f37d50d5c60ef1cea87357671 (patch) | |
tree | 9b75296d17a8d982e82f79dfdc3d75b3e0d8e9e7 | |
parent | f4fe6df6aa92573368c3fa0edb9fd03e64d9d059 (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.cxx | 6 |
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; } |