diff options
author | Dr. David Alan Gilbert <dave@treblig.org> | 2021-01-24 17:54:31 +0000 |
---|---|---|
committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2021-01-25 14:30:36 +0100 |
commit | 506036ba3f98db5fd83abaf494d3a9ab85561c18 (patch) | |
tree | 377a54b5e3dcdc7483aaa4221baa1fd104699130 /compilerplugins | |
parent | cc46ce1b78fe71eb66fc0a09e44f01428a9e4069 (diff) |
clang/constfieldsrewrite.cxx: Remove duped lines
Spotted by cppcheck.
A duplicated if and comment; looks like a merge screwup ?
Change-Id: Iceed88d85d42be601975b7a1daff7a1f002644d6
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/109863
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'compilerplugins')
-rw-r--r-- | compilerplugins/clang/constfieldsrewrite.cxx | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/compilerplugins/clang/constfieldsrewrite.cxx b/compilerplugins/clang/constfieldsrewrite.cxx index 03fb3d0c3609..52da9a153455 100644 --- a/compilerplugins/clang/constfieldsrewrite.cxx +++ b/compilerplugins/clang/constfieldsrewrite.cxx @@ -93,9 +93,6 @@ bool ConstFieldsRewrite::VisitFieldDecl(const FieldDecl* fieldDecl) // in case we've already processed this field if (fieldDecl->getType().isConstQualified()) return true; - // in case we've already processed this field - if (fieldDecl->getType().isConstQualified()) - return true; // TODO rewriting T& is a bit trickier if (loplugin::TypeCheck(fieldDecl->getType()).LvalueReference()) return true; |