comparison utilities/json-cli.cpp @ 11:aa61cb5c5754

Use mapper

 using namespace Vamp;
 using namespace Vamp::HostExt;
 using namespace json11;
 using namespace vampipe;
 
-static map<uint32_t, Plugin *> loadedPlugins;
-static set<uint32_t> initialisedPlugins;
-
-static uint32_t nextHandle = 1; // plugin handle type must fit in JSON number
+class Mapper : public PluginHandleMapper
+{
+public:
+    Mapper() : m_nextHandle(1) { }
+
+    void addPlugin(Plugin *p) {
+	if (m_rplugins.find(p) == m_rplugins.end()) {
+	    int32_t h = m_nextHandle++;
+	    m_plugins[h] = p;
+	    m_rplugins[p] = h;
+	}
+    }
+    
+    int32_t pluginToHandle(Plugin *p) {
+	if (m_rplugins.find(p) == m_rplugins.end()) {
+	    throw NotFound();
+	}
+	return m_rplugins[p];
+    }
+    
+    Plugin *handleToPlugin(int32_t h) {
+	if (m_plugins.find(h) == m_plugins.end()) {
+	    throw NotFound();
+	}
+	return m_plugins[h];
+    }
+
+    bool isInitialised(int32_t h) {
+	return m_initialisedPlugins.find(h) != m_initialisedPlugins.end();
+    }
+
+    void markInitialised(int32_t h) {
+	m_initialisedPlugins.insert(h);
+    }
+    
+private:
+//!!! + mutex
+    int32_t m_nextHandle; // plugin handle type must fit in JSON number
+    map<uint32_t, Plugin *> m_plugins;
+    map<Plugin *, uint32_t> m_rplugins;
+    set<uint32_t> m_initialisedPlugins;
+};
+
+static Mapper mapper;
 
 Vamp::HostExt::LoadResponse
 loadPlugin(json11::Json j) {
 
     auto req = VampJson::toLoadRequest(j);

     auto loadResponse = loadPlugin(j);
 
     if (!loadResponse.plugin) {
 	throw VampJson::Failure("plugin load failed");
     }
-    
-    uint32_t h = nextHandle++;
-    loadedPlugins[h] = loadResponse.plugin;
-
-    Json::object response;
-    response["pluginHandle"] = double(h);
-    response["staticData"] =
-	VampJson::fromPluginStaticData(loadResponse.staticData);
-    response["defaultConfiguration"] =
-	VampJson::fromPluginConfiguration(loadResponse.defaultConfiguration);
-
-    cerr << "Loaded plugin: handle is " << h << endl;
-    
-    return Json(response);
+
+    mapper.addPlugin(loadResponse.plugin);
+    
+    return VampJson::fromLoadResponse(loadResponse, mapper);
 }
 
 Json
 handle_configure(Json j)
 {

 		{ "pluginHandle", Json::NUMBER },
 		{ "configuration", Json::OBJECT }}, err)) {
 	throw VampJson::Failure("malformed configuration request: " + err);
     }
 
-    uint32_t handle = j["pluginHandle"].int_value();
-
-    if (loadedPlugins.find(handle) == loadedPlugins.end()) {
-	throw VampJson::Failure("unknown plugin handle");
-    }
-
-    if (initialisedPlugins.find(handle) != initialisedPlugins.end()) {
+    int32_t handle = j["pluginHandle"].int_value();
+
+    if (mapper.isInitialised(handle)) {
 	throw VampJson::Failure("plugin has already been initialised");
     }
 
-    Plugin *plugin = loadedPlugins[handle];
+    Plugin *plugin = mapper.handleToPlugin(handle);
     
     Json config = j["configuration"];
 
     configurePlugin(plugin, config);
 
-    initialisedPlugins.insert(handle);
+    mapper.markInitialised(handle);
 
     cerr << "Configured and initialised plugin " << handle << endl;
 
+    //!!! to VampJson:
     Json::object jout;
     Json::array outs;
     Vamp::Plugin::OutputList vouts = plugin->getOutputDescriptors();
     for (auto &o: vouts) {
 	outs.push_back(VampJson::fromOutputDescriptor(o));

 	try {
 	    Json result = handle(line);
 	    cout << success_response(result).dump() << endl;
 	} catch (const VampJson::Failure &e) {
 	    cout << error_response(e.what()).dump() << endl;
+	} catch (const PluginHandleMapper::NotFound &e) {
+	    cout << error_response(e.what()).dump() << endl;
 	}
     }
 
     return 0;
 }