diff vamp-server/simple-server.cpp @ 185:3eb00e5c76c4

Pull step & block size out into framing struct, return in config

Update the C++ code to separate out the framing parameters (step and
block size) from the configuration structure into their own structure,
as in the latest schema, and to return the accepted framing params in
the configuration response.

This also implies that the plugin stub (which adapts Piper API
back to Vamp) makes a note of the returned values,
making them available via its own getPreferredStep/BlockSize
so that the host can retry the initialise call in the case where it
failed for having the wrong values first time.

--- a/vamp-server/simple-server.cpp	Fri Feb 03 13:00:42 2017 +0000
+++ b/vamp-server/simple-server.cpp	Fri Feb 03 16:23:21 2017 +0000
@@ -484,11 +484,18 @@
             throw runtime_error("plugin has already been configured");
         }
 
+        if (creq.configuration.framing.stepSize == 0 ||
+            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()) {
             mapper.markConfigured
-                (h, creq.configuration.channelCount, creq.configuration.blockSize);
+                (h,
+                 creq.configuration.channelCount,
+                 creq.configuration.framing.blockSize);
             response.success = true;
         }
         break;