comparison vamp-server/simple-server.cpp @ 293:ae7397deffaa

Improve error handling (i.e. add some) for failure to initialise

         break;
 
     case RRType::Load:
         response.loadResponse =
             LoaderRequests().loadPlugin(request.loadRequest);
-        if (response.loadResponse.plugin != nullptr) {
-            mapper.addPlugin(response.loadResponse.plugin);
-            if (debug) {
-                cerr << "piper-vamp-server " << pid
-                     << ": loaded plugin, handle = "
-                     << mapper.pluginToHandle(response.loadResponse.plugin)
-                     << endl;
-            }
-            response.success = true;
-        }
+
+        if (!response.loadResponse.plugin) {
+            throw runtime_error("unable to load plugin");
+        }
+            
+        mapper.addPlugin(response.loadResponse.plugin);
+        if (debug) {
+            cerr << "piper-vamp-server " << pid
+                 << ": loaded plugin, handle = "
+                 << mapper.pluginToHandle(response.loadResponse.plugin)
+                 << endl;
+        }
+        response.success = true;
         break;
         
     case RRType::Configure:
     {
         auto &creq = request.configurationRequest;

             creq.configuration.framing.blockSize == 0) {
             throw runtime_error("step and block size must be non-zero");
         }
 
         response.configurationResponse = LoaderRequests().configurePlugin(creq);
+
+        if (response.configurationResponse.outputs.empty()) {
+            throw runtime_error("plugin failed to initialise");
+        }
         
-        if (!response.configurationResponse.outputs.empty()) {
-            mapper.markConfigured
-                (h,
-                 creq.configuration.channelCount,
-                 response.configurationResponse.framing.blockSize);
-            response.success = true;
-        }
+        mapper.markConfigured
+            (h,
+             creq.configuration.channelCount,
+             response.configurationResponse.framing.blockSize);
+        response.success = true;
         break;
     }
 
     case RRType::Process:
     {