diff options
| author | Noel Grandin <noel.grandin@collabora.co.uk> | 2018-01-31 16:29:17 +0200 |
|---|---|---|
| committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2018-02-02 07:39:50 +0100 |
| commit | ac7132598b1dfc4d266b89565755bd2795db9e0b (patch) | |
| tree | 360048822cb8b90ba93cc320d49211c8046f0ee3 /compilerplugins | |
| parent | 1d9868329658eab34352c499bc2893752cc3aaaf (diff) | |
loplugin:datamembershadow in PopupMenuToolbarController
this looks like an oversight, just use the baseclass member
Change-Id: Ida36be64266bbc33a6bd1d89dcb665d24a6dd83f
Reviewed-on: https://gerrit.libreoffice.org/49075
Reviewed-by: Maxim Monastirsky <momonasmon@gmail.com>
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Tested-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'compilerplugins')
| -rw-r--r-- | compilerplugins/clang/datamembershadow.cxx | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/compilerplugins/clang/datamembershadow.cxx b/compilerplugins/clang/datamembershadow.cxx index 210ee1c9a0d1..c4e9370b36f1 100644 --- a/compilerplugins/clang/datamembershadow.cxx +++ b/compilerplugins/clang/datamembershadow.cxx @@ -61,8 +61,6 @@ bool DataMemberShadow::VisitFieldDecl(FieldDecl const * fieldDecl) return true; if (loplugin::isSamePathname(aFileName, SRCDIR "/filter/source/graphicfilter/idxf/dxfentrd.hxx")) return true; - if (loplugin::isSamePathname(aFileName, SRCDIR "/framework/source/uielement/popuptoolbarcontroller.cxx")) - return true; if (loplugin::isSamePathname(aFileName, SRCDIR "/sc/source/ui/vba/vbastyles.hxx")) return true; if (loplugin::isSamePathname(aFileName, SRCDIR "/sd/inc/Outliner.hxx")) @@ -98,6 +96,8 @@ bool DataMemberShadow::VisitFieldDecl(FieldDecl const * fieldDecl) return false; for (const FieldDecl* baseFieldDecl : baseCXXRecordDecl->fields()) { + // TODO look for overlaps even with private fields + if (baseFieldDecl->getAccess() == AS_private || !baseFieldDecl->getDeclName().isIdentifier() || fieldDecl->getName() != baseFieldDecl->getName()) { |