compilerplugins/clang/unusedmethods.cxx | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)
New commits: commit b40e57f403b3d4ccb90f3b76caf8d537b61826dc Author: Noel Grandin <noel.gran...@collabora.co.uk> AuthorDate: Mon Sep 10 16:11:05 2018 +0200 Commit: Noel Grandin <noel.gran...@collabora.co.uk> CommitDate: Tue Sep 11 08:18:16 2018 +0200 fix loplugin:unusedmethods revert the part of commit 9f4d23c15115d64febd6bf01f870cc157badd350 Date: Mon Aug 13 17:24:26 2018 +0200 filter out some of the AST in the plugins that applied to this plugin. Turns out it really needs to see __all__ the code in order to produce good results. Change-Id: If580a701049d2570f2a833327b2189641090079b Reviewed-on: https://gerrit.libreoffice.org/60279 Reviewed-by: Noel Grandin <noel.gran...@collabora.co.uk> Tested-by: Noel Grandin <noel.gran...@collabora.co.uk> diff --git a/compilerplugins/clang/unusedmethods.cxx b/compilerplugins/clang/unusedmethods.cxx index 3837aa6d00ff..a84b7e8d237d 100644 --- a/compilerplugins/clang/unusedmethods.cxx +++ b/compilerplugins/clang/unusedmethods.cxx @@ -75,11 +75,11 @@ static std::set<MyFuncInfo> calledFromOutsideSet; class UnusedMethods: - public loplugin::FilteringPlugin<UnusedMethods> + public RecursiveASTVisitor<UnusedMethods>, public loplugin::Plugin { public: explicit UnusedMethods(loplugin::InstantiationData const & data): - FilteringPlugin(data) {} + Plugin(data) {} virtual void run() override { _______________________________________________ Libreoffice-commits mailing list libreoffice-comm...@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits