clang-tidy modernize-pass-by-value fix

daniele77 · daniele77 · commit 3663d96705d0 · 2021-03-18T18:26:31.000+01:00
diff --git a/examples/pluginmanager.cpp b/examples/pluginmanager.cpp
@@ -44,6 +44,7 @@
 
 #include "cli/clilocalsession.h"
 #include "cli/cli.h"
+#include <utility>
 #include <vector>
 
 using namespace cli;
@@ -55,7 +56,7 @@ using namespace std;
 class Plugin
 {
 public:
-    Plugin(const string& _name) : name(_name) { cout << "Plugin " << name << " loaded" << endl; }
+    Plugin(string _name) : name(std::move(_name)) { cout << "Plugin " << name << " loaded" << endl; }
     virtual ~Plugin() { cout << "Plugin " << name << " unloaded" << endl; }
     const string& Name() const { return name; }
 private:
diff --git a/include/cli/cli.h b/include/cli/cli.h
@@ -30,7 +30,6 @@
 #ifndef CLI_H_
 #define CLI_H_
 
-#include <iostream>
 #include <string>
 #include <vector>
 #include <memory>
@@ -44,6 +43,8 @@
 #include "detail/fromstring.h"
 #include "historystorage.h"
 #include "volatilehistorystorage.h"
+#include <iostream>
+#include <utility>
 
 namespace cli
 {
@@ -128,7 +129,7 @@ namespace cli
         ) :
             globalHistoryStorage(std::move(historyStorage)),
             rootMenu(std::move(_rootMenu)),
-            exitAction(_exitAction)
+            exitAction(std::move(_exitAction))
         {
         }
 
@@ -191,7 +192,7 @@ namespace cli
     class Command
     {
     public:
-        explicit Command(const std::string& _name) : name(_name), enabled(true) {}
+        explicit Command(std::string _name) : name(std::move(_name)), enabled(true) {}
         virtual ~Command() = default;
         virtual void Enable() { enabled = true; }
         virtual void Disable() { enabled = false; }
@@ -313,9 +314,9 @@ namespace cli
     private:
         struct Descriptor
         {
-            Descriptor() {}
-            Descriptor(const std::weak_ptr<Command>& c, const std::weak_ptr<CmdVec>& v) :
-                cmd(c), cmds(v)
+            Descriptor() = default;
+            Descriptor(std::weak_ptr<Command> c, std::weak_ptr<CmdVec> v) :
+                cmd(std::move(c)), cmds(std::move(v))
             {}
             void Enable()
             {
@@ -359,8 +360,8 @@ namespace cli
 
         Menu() : Command({}), parent(nullptr), description(), cmds(std::make_shared<Cmds>()) {}
 
-        Menu(const std::string& _name, const std::string& desc = "(menu)") :
-            Command(_name), parent(nullptr), description(desc), cmds(std::make_shared<Cmds>())
+        Menu(const std::string& _name, std::string  desc = "(menu)") :
+            Command(_name), parent(nullptr), description(std::move(desc)), cmds(std::make_shared<Cmds>())
         {}
 
         template <typename F>
@@ -568,10 +569,10 @@ namespace cli
         VariadicFunctionCommand(
             const std::string& _name,
             F fun,
-            const std::string& desc,
-            const std::vector<std::string>& parDesc
+            std::string desc,
+            std::vector<std::string> parDesc
         )
-            : Command(_name), func(std::move(fun)), description(desc), parameterDesc(parDesc)
+            : Command(_name), func(std::move(fun)), description(std::move(desc)), parameterDesc(std::move(parDesc))
         {
         }
 
@@ -626,10 +627,10 @@ namespace cli
         FreeformCommand(
             const std::string& _name,
             F fun,
-            const std::string& desc,
-            const std::vector<std::string>& parDesc
+            std::string desc,
+            std::vector<std::string> parDesc
         )
-            : Command(_name), func(std::move(fun)), description(desc), parameterDesc(parDesc)
+            : Command(_name), func(std::move(fun)), description(std::move(desc)), parameterDesc(std::move(parDesc))
         {
         }
 
diff --git a/include/cli/detail/split.h b/include/cli/detail/split.h
@@ -30,9 +30,10 @@
 #ifndef CLI_DETAIL_SPLIT_H_
 #define CLI_DETAIL_SPLIT_H_
 
+#include <algorithm>
 #include <string>
+#include <utility>
 #include <vector>
-#include <algorithm>
 
 namespace cli
 {
@@ -42,7 +43,7 @@ namespace detail
 class Text
 {
 public:
-    explicit Text(const std::string& _input) : input(_input)
+    explicit Text(std::string _input) : input(std::move(_input))
     {
     }
     void SplitInto(std::vector<std::string>& strs)
diff --git a/include/cli/filehistorystorage.h b/include/cli/filehistorystorage.h
@@ -32,16 +32,17 @@
 
 #include "historystorage.h"
 #include <fstream>
+#include <utility>
 
 namespace cli
 {
 
 class FileHistoryStorage : public HistoryStorage
 {
 public:
-    FileHistoryStorage(const std::string& _fileName, std::size_t size = 1000) : 
+    FileHistoryStorage(std::string _fileName, std::size_t size = 1000) : 
         maxSize(size),
-        fileName(_fileName)
+        fileName(std::move(_fileName))
     {
     }
     void Store(const std::vector<std::string>& cmds) override

Original file line number	Diff line number	Diff line change
`@@ -30,9 +30,10 @@`
`30`	`30`	`#ifndef CLI_DETAIL_SPLIT_H_`
`31`	`31`	`#define CLI_DETAIL_SPLIT_H_`
`32`	`32`
	`33`	`+#include <algorithm>`
`33`	`34`	`#include <string>`
	`35`	`+#include <utility>`
`34`	`36`	`#include <vector>`
`35`		`-#include <algorithm>`
`36`	`37`
`37`	`38`	`namespace cli`
`38`	`39`	`{`
`@@ -42,7 +43,7 @@ namespace detail`
`42`	`43`	`class Text`
`43`	`44`	`{`
`44`	`45`	`public:`
`45`		`- explicit Text(const std::string& _input) : input(_input)`
	`46`	`+ explicit Text(std::string _input) : input(std::move(_input))`
`46`	`47`	`{`
`47`	`48`	`}`
`48`	`49`	`void SplitInto(std::vector<std::string>& strs)`
Original file line number	Diff line number	Diff line change
`@@ -32,16 +32,17 @@`
`32`	`32`
`33`	`33`	`#include "historystorage.h"`
`34`	`34`	`#include <fstream>`
	`35`	`+#include <utility>`
`35`	`36`
`36`	`37`	`namespace cli`
`37`	`38`	`{`
`38`	`39`
`39`	`40`	`class FileHistoryStorage : public HistoryStorage`
`40`	`41`	`{`
`41`	`42`	`public:`
`42`		`- FileHistoryStorage(const std::string& _fileName, std::size_t size = 1000) :`
	`43`	`+ FileHistoryStorage(std::string _fileName, std::size_t size = 1000) :`
`43`	`44`	`maxSize(size),`
`44`		`- fileName(_fileName)`
	`45`	`+ fileName(std::move(_fileName))`
`45`	`46`	`{`
`46`	`47`	`}`
`47`	`48`	`void Store(const std::vector<std::string>& cmds) override`