options: use move instead of copy in build_options()

Signed-off-by: Casey Bodley <cbodley@redhat.com>

diff --git a/src/common/options.cc b/src/common/options.cc
index 7de4ee241cc919b0568d625e9b88ddf8ed6ef83e..7ee706fc1fbe5daa71ddf1669bb8a682e08a9f92 100644 (file)
--- a/src/common/options.cc
+++ b/src/common/options.cc
@@ -6049,10 +6049,9 @@ static std::vector<Option> build_options()
   std::vector<Option> result = get_global_options();
 
   auto ingest = [&result](std::vector<Option>&& options, const char* svc) {
-    for (const auto &o_in : options) {
-      Option o(o_in);
+    for (auto &o : options) {
       o.add_service(svc);
-      result.push_back(o);
+      result.push_back(std::move(o));
     }
   };