change Server::listSource()

src/pvxs/server.h

@@ -102,7 +102,7 @@ public:
                                       int order =0);
 
     //! List all source names and priorities.
-    void listSource(std::vector<std::pair<std::string, int> >& names);
+    std::vector<std::pair<std::string, int> > listSource();
 
     explicit operator bool() const { return !!pvt; }
 

src/server.cpp

@@ -118,12 +118,11 @@ std::shared_ptr<Source> Server::getSource(const std::string& name, int order)
     return ret;
 }
 
-void Server::listSource(std::vector<std::pair<std::string, int> > &names)
+std::vector<std::pair<std::string, int> > Server::listSource()
 {
     if(!pvt)
         throw std::logic_error("NULL Server");
-
-    names.clear();
+    std::vector<std::pair<std::string, int> > names;
 
     auto G(pvt->sourcesLock.lockReader());
 
@@ -132,6 +131,8 @@ void Server::listSource(std::vector<std::pair<std::string, int> > &names)
     for(auto& pair : pvt->sources) {
         names.emplace_back(pair.first.second, pair.first.first);
     }
+
+    return names;
 }
 
 const Config& Server::config() const