From fd2bac3e22e6b1454c5220d1b5a1ab7844a2d39b Mon Sep 17 00:00:00 2001 From: Dennis Klein Date: Wed, 11 Dec 2019 15:21:00 +0100 Subject: [PATCH] Modernize ctor https://clang.llvm.org/extra/clang-tidy/checks/modernize-pass-by-value.html#pass-by-value-in-constructors --- fairmq/Plugin.cxx | 17 +++++++++++------ fairmq/Plugin.h | 8 ++++++-- 2 files changed, 17 insertions(+), 8 deletions(-) diff --git a/fairmq/Plugin.cxx b/fairmq/Plugin.cxx index a814105b..9ed1256d 100644 --- a/fairmq/Plugin.cxx +++ b/fairmq/Plugin.cxx @@ -1,5 +1,5 @@ /******************************************************************************** - * Copyright (C) 2017 GSI Helmholtzzentrum fuer Schwerionenforschung GmbH * + * Copyright (C) 2017-2019 GSI Helmholtzzentrum fuer Schwerionenforschung GmbH * * * * This software is distributed under the terms of the * * GNU Lesser General Public Licence (LGPL) version 3, * @@ -8,15 +8,20 @@ #include #include +#include using namespace std; -fair::mq::Plugin::Plugin(const string name, const Version version, const string maintainer, const string homepage, PluginServices* pluginServices) - : fkName{name} +fair::mq::Plugin::Plugin(string name, + Version version, + string maintainer, + string homepage, + PluginServices* pluginServices) + : fkName(std::move(name)) , fkVersion(version) - , fkMaintainer{maintainer} - , fkHomepage{homepage} - , fPluginServices{pluginServices} + , fkMaintainer(std::move(maintainer)) + , fkHomepage(std::move(homepage)) + , fPluginServices(pluginServices) { LOG(debug) << "Loaded plugin: " << *this; } diff --git a/fairmq/Plugin.h b/fairmq/Plugin.h index 43258441..cd7d70bc 100644 --- a/fairmq/Plugin.h +++ b/fairmq/Plugin.h @@ -1,5 +1,5 @@ /******************************************************************************** - * Copyright (C) 2017 GSI Helmholtzzentrum fuer Schwerionenforschung GmbH * + * Copyright (C) 2017-2019 GSI Helmholtzzentrum fuer Schwerionenforschung GmbH * * * * This software is distributed under the terms of the * * GNU Lesser General Public Licence (LGPL) version 3, * @@ -44,7 +44,11 @@ class Plugin using Version = tools::Version; Plugin() = delete; - Plugin(const std::string name, const Version version, const std::string maintainer, const std::string homepage, PluginServices* pluginServices); + Plugin(std::string name, + Version version, + std::string maintainer, + std::string homepage, + PluginServices* pluginServices); Plugin(const Plugin&) = delete; Plugin operator=(const Plugin&) = delete;