diff options
| author | Noel Grandin <noel.grandin@collabora.co.uk> | 2018-09-10 16:11:05 +0200 |
|---|---|---|
| committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2018-09-11 08:18:16 +0200 |
| commit | b40e57f403b3d4ccb90f3b76caf8d537b61826dc (patch) | |
| tree | 8161907a53f6cf1921bdf4a0da0986398a26b0f4 | |
| parent | 1476d95b6ed3afa35cccc46f61865ea938b62144 (diff) | |
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.grandin@collabora.co.uk>
Tested-by: Noel Grandin <noel.grandin@collabora.co.uk>
| -rw-r--r-- | compilerplugins/clang/unusedmethods.cxx | 4 |
1 files changed, 2 insertions, 2 deletions
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 { |