[alpha-unito#71] solved performance issues (minor fix pt2)

ARMartinelli · Feb 26, 2019 · aad4b85 · aad4b85
1 parent 2b210f5
commit aad4b85
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 6 deletions.
diff --git a/include/pico/Operators/InOut/ReadFromSocket.hpp b/include/pico/Operators/InOut/ReadFromSocket.hpp
@@ -62,8 +62,7 @@ class ReadFromSocket : public InputOperator<std::string> {
    * Copy constructor.
    */
   ReadFromSocket(const ReadFromSocket &copy)
-      : InputOperator<std::string>(copy) {
-    server_name = copy.server_name;
+      : InputOperator<std::string>(copy), server_name(copy.server_name) {
     port = copy.port;
     delimiter = copy.delimiter;
   }

diff --git a/include/pico/Operators/InOut/WriteToDisk.hpp b/include/pico/Operators/InOut/WriteToDisk.hpp
@@ -57,7 +57,7 @@ class WriteToDisk : public OutputOperator<In> {
    *
    * Creates a new WriteToDisk operator by defining its kernel function.
    */
-  WriteToDisk(const std::string fname_, std::function<std::string(In)> func_)
+  WriteToDisk(const std::string& fname_, std::function<std::string(In)> func_)
       : OutputOperator<In>(StructureType::BAG),
         fname(fname_),  //
         usr_func(true),
@@ -70,7 +70,7 @@ class WriteToDisk : public OutputOperator<In> {
    *
    * Creates a new WriteToDisk writing by ostream.
    */
-  WriteToDisk(const std::string fname_)
+  WriteToDisk(const std::string& fname_)
       : OutputOperator<In>(StructureType::BAG), fname(fname_) {}
 
   /**

diff --git a/include/pico/Operators/InOut/WriteToStdOut.hpp b/include/pico/Operators/InOut/WriteToStdOut.hpp
@@ -72,8 +72,7 @@ class WriteToStdOut : public OutputOperator<In> {
   /**
    * Copy constructor.
    */
-  WriteToStdOut(const WriteToStdOut& copy) : OutputOperator<In>(copy) {
-    func = copy.func;
+  WriteToStdOut(const WriteToStdOut& copy) : OutputOperator<In>(copy), func(copy.func) {
     usr_func = copy.usr_func;
   }