diff options
author | Stephan Bergmann <sbergman@redhat.com> | 2017-10-11 10:45:27 +0200 |
---|---|---|
committer | Stephan Bergmann <sbergman@redhat.com> | 2017-10-11 14:26:44 +0200 |
commit | 948c9c0a313d932d22cf275792077862eb4de334 (patch) | |
tree | bd52b5ab63d4d84a5731429acc12fb5f46f75b55 /compilerplugins/clang/dyncastvisibility.cxx | |
parent | 10ba813b349fc7db3a2304ab9d31cccd06039cc6 (diff) |
Use unique class names for better FlameGraph results
...of measuring loplugin performance, when gathering data with perf and all
plugins that used "Visitor" as class name were lumped together. (Cf.
<https://whatofhow.wordpress.com/2016/06/28/plugin-flamed/>.)
Change-Id: Ie482f443faced7469528da4772e735bc2eda596d
Reviewed-on: https://gerrit.libreoffice.org/43324
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
Tested-by: Stephan Bergmann <sbergman@redhat.com>
Diffstat (limited to 'compilerplugins/clang/dyncastvisibility.cxx')
-rw-r--r-- | compilerplugins/clang/dyncastvisibility.cxx | 9 |
1 files changed, 5 insertions, 4 deletions
diff --git a/compilerplugins/clang/dyncastvisibility.cxx b/compilerplugins/clang/dyncastvisibility.cxx index 8cae22fbe534..e6e1f6eb79df 100644 --- a/compilerplugins/clang/dyncastvisibility.cxx +++ b/compilerplugins/clang/dyncastvisibility.cxx @@ -75,11 +75,11 @@ StringRef vis(Visibility v) { } } -class Visitor final: - public RecursiveASTVisitor<Visitor>, public loplugin::Plugin +class DynCastVisibility final: + public RecursiveASTVisitor<DynCastVisibility>, public loplugin::Plugin { public: - explicit Visitor(InstantiationData const & data): Plugin(data) {} + explicit DynCastVisibility(InstantiationData const & data): Plugin(data) {} bool shouldVisitTemplateInstantiations() const { return true; } @@ -156,7 +156,8 @@ private: } }; -static loplugin::Plugin::Registration<Visitor> reg("dyncastvisibility"); +static loplugin::Plugin::Registration<DynCastVisibility> reg( + "dyncastvisibility"); } |