comparison plugin/PiperVampPluginFactory.cpp @ 1375:da5f4d37988d dev/refactor-piper-related

Invert control, injecting a server path on construction. Also add initializer_list constructor for multiple servers.

     void log(std::string message) const override {
         SVDEBUG << "PiperVampPluginFactory: " << message << endl;
     }
 };
 
+PiperVampPluginFactory::PiperVampPluginFactory(std::initializer_list<QString> servers) :
+    m_logger(new Logger)
+{
+    HelperExecPath hep(HelperExecPath::AllInstalled);
+
+    for (auto server: servers) {
+        for (auto platformHelper: hep.getHelperExecutables(server))
+            m_servers.push_back(platformHelper);
+    }
+
+    for (auto n: m_servers) {
+        SVDEBUG << "NOTE: PiperVampPluginFactory: Found server: " 
+                << n.executable << endl;
+    }
+
+    if (m_servers.empty()) {
+        SVDEBUG << "NOTE: No Piper Vamp servers found in installation;" 
+                << " found none of the following:" << endl;
+        for (auto serverName: servers)
+            for (auto d: hep.getHelperCandidatePaths(serverName)) {
+                SVDEBUG << "NOTE: " << d << endl;
+            }
+    }
+}
+
 PiperVampPluginFactory::PiperVampPluginFactory() :
-    m_logger(new Logger)
-{
-    QString serverName = "piper-vamp-simple-server";
-
-    HelperExecPath hep(HelperExecPath::AllInstalled);
-    m_servers = hep.getHelperExecutables(serverName);
-
-    for (auto n: m_servers) {
-        SVDEBUG << "NOTE: PiperVampPluginFactory: Found server: "
-                << n.executable << endl;
-    }
-    
-    if (m_servers.empty()) {
-        SVDEBUG << "NOTE: No Piper Vamp servers found in installation;"
-                << " found none of the following:" << endl;
-        for (auto d: hep.getHelperCandidatePaths(serverName)) {
-            SVDEBUG << "NOTE: " << d << endl;
-        }
-    }
-}
+    PiperVampPluginFactory({"piper-vamp-simple-server"}) {}
 
 PiperVampPluginFactory::~PiperVampPluginFactory()
 {
     delete m_logger;
 }