summaryrefslogtreecommitdiff
path: root/compilerplugins/clang/passsequencebyref.cxx
diff options
context:
space:
mode:
authorNoel Grandin <noel@peralex.com>2014-05-16 11:18:12 +0200
committerNoel Grandin <noel@peralex.com>2014-05-16 11:18:45 +0200
commit97d50c425f83f05bb395956e1f855db00b979f27 (patch)
treee2b0334db5dffa515bd14563ead968b3d970065a /compilerplugins/clang/passsequencebyref.cxx
parent0bc869459b8dfc05923249ec07ccfd4f0ddd2c2c (diff)
combine the pass-by-ref clang plugins
Change-Id: Idac24afb7cb67fa2d539553fb9fa049c2d61ecf0
Diffstat (limited to 'compilerplugins/clang/passsequencebyref.cxx')
-rw-r--r--compilerplugins/clang/passsequencebyref.cxx63
1 files changed, 0 insertions, 63 deletions
diff --git a/compilerplugins/clang/passsequencebyref.cxx b/compilerplugins/clang/passsequencebyref.cxx
deleted file mode 100644
index 851307bb2710..000000000000
--- a/compilerplugins/clang/passsequencebyref.cxx
+++ /dev/null
@@ -1,63 +0,0 @@
-/* -*- 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: */