diff options
-rw-r--r-- | compilerplugins/clang/redundantcast.cxx | 5 | ||||
-rw-r--r-- | compilerplugins/clang/test/redundantcast.cxx | 4 |
2 files changed, 0 insertions, 9 deletions
diff --git a/compilerplugins/clang/redundantcast.cxx b/compilerplugins/clang/redundantcast.cxx index c28878e9e536..c86d7dc1ab73 100644 --- a/compilerplugins/clang/redundantcast.cxx +++ b/compilerplugins/clang/redundantcast.cxx @@ -682,11 +682,6 @@ bool RedundantCast::VisitCXXDynamicCastExpr(CXXDynamicCastExpr const * expr) { if (ignoreLocation(expr)) { return true; } - // ignore dynamic_cast<T1>(static_cast<T1>(void*)), it's necessary - if (auto subStaticCast = dyn_cast<CXXStaticCastExpr>(expr->getSubExpr())) { - if (loplugin::TypeCheck(subStaticCast->getSubExpr()->getType()).Pointer().Void()) - return true; - } // so far this only deals with dynamic casting from T to T auto const sub = compat::getSubExprAsWritten(expr); auto const t1 = expr->getTypeAsWritten(); diff --git a/compilerplugins/clang/test/redundantcast.cxx b/compilerplugins/clang/test/redundantcast.cxx index 38c44f837cbc..20578079c2cb 100644 --- a/compilerplugins/clang/test/redundantcast.cxx +++ b/compilerplugins/clang/test/redundantcast.cxx @@ -333,15 +333,11 @@ void testDynamicCast() { S1 * s1 = nullptr; S2 * s2 = nullptr; - void * v1 = nullptr; (void) dynamic_cast<S2 *>(s1); (void) dynamic_cast<S1 *>(s2); (void) dynamic_cast<S2 *>(s2); // expected-error {{redundant dynamic cast from 'S2 *' to 'S2 *' [loplugin:redundantcast]}} (void) dynamic_cast<S3 *>(s2); - // used in some assert in vcl - (void) dynamic_cast<S1 *>(static_cast<S1*>(v1)); - (void) dynamic_cast<S2 *>(static_cast<S2*>(s1)); // expected-error {{redundant dynamic cast from 'S2 *' to 'S2 *' [loplugin:redundantcast]}} } int main() { |