summaryrefslogtreecommitdiff
path: root/compilerplugins
diff options
context:
space:
mode:
authorNoel Grandin <noel@peralex.com>2014-05-14 09:58:40 +0200
committerNoel Grandin <noelgrandin@gmail.com>2014-05-14 08:10:22 +0000
commit248145f99e95cc30bb6231a8e5ea4e294f147040 (patch)
treec3b5c1e226cc631cde530f2cfb7df2d46f00b8c9 /compilerplugins
parent671eb12dee290607ed66f3b325f28e7bd4695cba (diff)
Find places where uno::Sequence is passed by value.
Implement a clang plugin to find them, and clean up existing code to pass them by reference. Change-Id: If642d87407c73346d9c0164b9fc77c5c3c4354b8 Reviewed-on: https://gerrit.libreoffice.org/9351 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
Diffstat (limited to 'compilerplugins')
-rw-r--r--compilerplugins/clang/passsequencebyref.cxx63
1 files changed, 63 insertions, 0 deletions
diff --git a/compilerplugins/clang/passsequencebyref.cxx b/compilerplugins/clang/passsequencebyref.cxx
new file mode 100644
index 000000000000..851307bb2710
--- /dev/null
+++ b/compilerplugins/clang/passsequencebyref.cxx
@@ -0,0 +1,63 @@
+/* -*- Mode: C++; tab-width: 4; indent-tabs-mode: nil; c-basic-offset: 4 -*- */
+/*
+ * 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 <string>
+
+#include "plugin.hxx"
+
+// Find places where com::sun::star::uno::Sequence is passed by value.
+// It's not very efficient, because that copies a whole list
+// They should rather be passed by reference.
+
+namespace {
+
+class PassSequenceByRef:
+ public RecursiveASTVisitor<PassSequenceByRef>, public loplugin::Plugin
+{
+public:
+ explicit PassSequenceByRef(InstantiationData const & data): Plugin(data) {}
+
+ virtual void run() override { TraverseDecl(compiler.getASTContext().getTranslationUnitDecl()); }
+
+ bool VisitFunctionDecl(const FunctionDecl * decl);
+};
+
+bool PassSequenceByRef::VisitFunctionDecl(const FunctionDecl * functionDecl) {
+ if (ignoreLocation(functionDecl)) {
+ return true;
+ }
+ // only warn on the definition/prototype of the function,
+ // not on the function implementation
+ if (functionDecl->isThisDeclarationADefinition() && functionDecl->getPreviousDecl() != nullptr) {
+ return true;
+ }
+ unsigned n = functionDecl->getNumParams();
+ for (unsigned i = 0; i != n; ++i) {
+ const ParmVarDecl * pvDecl = functionDecl->getParamDecl(i);
+ QualType t1 { pvDecl->getType() };
+ if (!t1->isClassType()) {
+ continue;
+ }
+ string typeName = t1.getUnqualifiedType().getCanonicalType().getAsString();
+ if (typeName.find("class com::sun::star::uno::Sequence") == 0) {
+ report(
+ DiagnosticsEngine::Warning,
+ "passing css::uno::Sequence by value, rather pass by reference .e.g. 'const css::uno::Sequence&' " + typeName,
+ pvDecl->getSourceRange().getBegin())
+ << pvDecl->getSourceRange();
+ }
+ }
+ return true;
+}
+
+loplugin::Plugin::Registration< PassSequenceByRef > X("passsequencebyref");
+
+}
+
+/* vim:set shiftwidth=4 softtabstop=4 expandtab: */