summaryrefslogtreecommitdiffstats
path: root/compilerplugins/clang/redundantfcast.cxx
diff options
context:
space:
mode:
authorNoel Grandin <noel.grandin@collabora.co.uk>2018-02-22 08:55:08 +0200
committerNoel Grandin <noel.grandin@collabora.co.uk>2018-02-22 08:56:29 +0200
commit50d0ec571f302fe54ac7ddac827b571c94554bec (patch)
tree92f630d1bfd8feb9cc1814e7b9a7b9db00b7a66a /compilerplugins/clang/redundantfcast.cxx
parentMerely forward-declare NotebookBar in vcl/syswin.hxx (diff)
downloadcore-50d0ec571f302fe54ac7ddac827b571c94554bec.tar.gz
core-50d0ec571f302fe54ac7ddac827b571c94554bec.zip
rename redundantcopy loplugin to redundantfcast
Change-Id: I34e28a30a4f1fd264c18c901cd94094531543271
Diffstat (limited to 'compilerplugins/clang/redundantfcast.cxx')
-rw-r--r--compilerplugins/clang/redundantfcast.cxx61
1 files changed, 61 insertions, 0 deletions
diff --git a/compilerplugins/clang/redundantfcast.cxx b/compilerplugins/clang/redundantfcast.cxx
new file mode 100644
index 000000000000..87d656c6d237
--- /dev/null
+++ b/compilerplugins/clang/redundantfcast.cxx
@@ -0,0 +1,61 @@
+/* -*- Mode: C++; tab-width: 4; indent-tabs-mode: nil; c-basic-offset: 4; fill-column: 100 -*- */
+/*
+ * This file is part of the LibreOffice project.
+ *
+ * This Source Code Form is subject to the terms of the Mozilla Public
+ * License, v. 2.0. If a copy of the MPL was not distributed with this
+ * file, You can obtain one at http://mozilla.org/MPL/2.0/.
+ */
+
+#include "check.hxx"
+#include "compat.hxx"
+#include "plugin.hxx"
+
+namespace
+{
+class RedundantFCast final : public RecursiveASTVisitor<RedundantFCast>, public loplugin::Plugin
+{
+public:
+ explicit RedundantFCast(loplugin::InstantiationData const& data)
+ : Plugin(data)
+ {
+ }
+
+ bool VisitCXXFunctionalCastExpr(CXXFunctionalCastExpr const* expr)
+ {
+ if (ignoreLocation(expr))
+ {
+ return true;
+ }
+ auto const t1 = expr->getTypeAsWritten();
+ auto const t2 = compat::getSubExprAsWritten(expr)->getType();
+ if (t1.getCanonicalType().getTypePtr() != t2.getCanonicalType().getTypePtr())
+ {
+ return true;
+ }
+ auto tc = loplugin::TypeCheck(t1);
+ if (!(tc.Class("OUString").Namespace("rtl").GlobalNamespace()
+ || tc.Class("Color").GlobalNamespace() || tc.Class("unique_ptr").StdNamespace()))
+ {
+ return true;
+ }
+ report(DiagnosticsEngine::Warning, "redundant functional cast from %0 to %1",
+ expr->getExprLoc())
+ << t2 << t1 << expr->getSourceRange();
+ return true;
+ }
+
+private:
+ void run() override
+ {
+ if (compiler.getLangOpts().CPlusPlus)
+ {
+ TraverseDecl(compiler.getASTContext().getTranslationUnitDecl());
+ }
+ }
+};
+
+static loplugin::Plugin::Registration<RedundantFCast> reg("redundantfcast");
+}
+
+/* vim:set shiftwidth=4 softtabstop=4 expandtab cinoptions=b1,g0,N-s cinkeys+=0=break: */