diff options
| author | Noel Grandin <noel.grandin@collabora.co.uk> | 2017-04-06 08:59:07 +0200 |
|---|---|---|
| committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2017-04-06 07:47:30 +0000 |
| commit | 7662e92c64ec194a2089f633a363d9dc45a4aa9d (patch) | |
| tree | 1086154f31c0c9c6386b44ab83bd382f5308df8c /compilerplugins | |
| parent | 823a8f3117a8be1de92168226f561ed804db3e93 (diff) | |
loplugin:redundantcast find cstyle double casts
Change-Id: I5507be190dac781e5cdb545a60acf3d50056c9f8
Reviewed-on: https://gerrit.libreoffice.org/36187
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'compilerplugins')
| -rw-r--r-- | compilerplugins/clang/redundantcast.cxx | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/compilerplugins/clang/redundantcast.cxx b/compilerplugins/clang/redundantcast.cxx index 7b3b05d8aced..845876425c7e 100644 --- a/compilerplugins/clang/redundantcast.cxx +++ b/compilerplugins/clang/redundantcast.cxx @@ -306,8 +306,8 @@ bool RedundantCast::VisitCStyleCastExpr(CStyleCastExpr const * expr) { << t1 << t2 << expr->getSourceRange(); return true; } - bool bBuiltinTypeBool = t1->isSpecificBuiltinType(BuiltinType::Bool); - if ((bBuiltinTypeBool || loplugin::TypeCheck(t1).Typedef()) && t1 == t2) + bool bBuiltinType = t1->isSpecificBuiltinType(BuiltinType::Bool) || t1->isSpecificBuiltinType(BuiltinType::Double); + if ((bBuiltinType || loplugin::TypeCheck(t1).Typedef()) && t1 == t2) { // Ignore FD_ISSET expanding to "...(SOCKET)(fd)..." in some Microsoft // winsock2.h (TODO: improve heuristic of determining that the whole |