Changeset 4096

Timestamp:

Sep 17, 2021, 1:24:28 AM (2 years ago)

Author:

Peter

Message:

simplify code and alter private interface to reflect what's going on inside

File:

• trunk/yat/utility/getvector.h (modified) (3 diffs)

trunk/yat/utility/getvector.h

@@ -30 +30 @@
 #include <istream>
 #include <string>
+#include <utility>
 #include <vector>
 
@@ -68 +69 @@
 
     template<typename T>
-    void convert(std::vector<std::string>& vec,
+    void convert(std::vector<std::string>&& vec,
                  std::vector<T>& result, boost::true_type is_string)
     {
-      // swap only swaps couple pointers, hence almost free. Do not us
-      // vec after this point.
-      std::swap(vec, result);
+      result = std::move(vec);
     }
 
 
     template<typename T>
-    void convert(std::vector<std::string>& vec,
+    void convert(const std::vector<std::string>& vec,
                  std::vector<T>& result, boost::false_type is_string)
     {
@@ -101 +100 @@
       split(vec, line, delim);
       typename boost::is_same<T, std::string> is_string;
-      convert(vec, result, is_string);
+      convert(std::move(vec), result, is_string);
       return is;
     }