mgr/ActivePyModule: return std::string_view instead of std::string copy

This impliciy heap allocation is unnecessary.

Signed-off-by: Max Kellermann <[email protected]>

Author: MaxKellermann

src/mgr/ActivePyModule.h

@@ -97,12 +97,12 @@ class ActivePyModule : public PyModuleRunner
     uri = str;
   }
 
-  std::string get_uri() const
+  std::string_view get_uri() const
   {
     return uri;
   }
 
-  std::string get_fin_thread_name() const
+  std::string_view get_fin_thread_name() const
   {
     return fin_thread_name;
   }

src/mgr/ActivePyModules.cc

@@ -770,9 +770,9 @@ std::map<std::string, std::string> ActivePyModules::get_services() const
   std::map<std::string, std::string> result;
   std::lock_guard l(lock);
   for (const auto& [name, module] : modules) {
-    std::string svc_str = module->get_uri();
+    const std::string_view svc_str = module->get_uri();
     if (!svc_str.empty()) {
-      result[name] = svc_str;
+      result.emplace(name, svc_str);
     }
   }