From cd8c6ffd87e8c53724d608c4084305c67c371548 Mon Sep 17 00:00:00 2001 From: Bent Bisballe Nyeng Date: Sun, 3 Feb 2019 12:58:56 +0100 Subject: Add metadata support to xml parser and dom model. RE-add unit-tests for version 1 instruments. --- src/DGDOM.h | 26 +++++++++++++++-- src/dgxmlparser.cc | 83 ++++++++++++++++++++++++++++++++++++++++++++++-------- src/domloader.cc | 4 +-- src/instrument.h | 8 ++---- src/rangemap.h | 2 +- src/sample.h | 2 -- 6 files changed, 100 insertions(+), 25 deletions(-) (limited to 'src') diff --git a/src/DGDOM.h b/src/DGDOM.h index 933c250..dea4228 100644 --- a/src/DGDOM.h +++ b/src/DGDOM.h @@ -103,13 +103,35 @@ struct InstrumentRefDOM std::vector channel_map; }; -struct DrumkitDOM +struct ClickMapDOM +{ + std::string instrument; + std::string colour; +}; + +struct MetadataDOM { - std::string name; std::string version; + std::string title; std::string description; + std::string license; + std::string notes; + std::string author; + std::string email; + std::string website; + std::string logo; + std::string image; + std::string image_map; + std::vector clickmaps; +}; + +struct DrumkitDOM +{ + std::string version; double samplerate; + MetadataDOM metadata; + std::vector instruments; std::vector channels; }; diff --git a/src/dgxmlparser.cc b/src/dgxmlparser.cc index fc5dbb9..28e183a 100644 --- a/src/dgxmlparser.cc +++ b/src/dgxmlparser.cc @@ -80,7 +80,8 @@ template static bool attrcpy(T& dest, const pugi::xml_node& src, const std::string& attr, bool opt = false) { const char* val = src.attribute(attr.c_str()).as_string(nullptr); - if(!val) { + if(!val) + { if(!opt) { ERR(dgxmlparser, "Attribute %s not found in %s, offset %d\n", @@ -89,7 +90,8 @@ static bool attrcpy(T& dest, const pugi::xml_node& src, const std::string& attr, return opt; } - if(!assign(dest, std::string(val))) { + if(!assign(dest, std::string(val))) + { ERR(dgxmlparser, "Attribute %s could not be assigned, offset %d\n", attr.data(), (int)src.offset_debug()); return false; @@ -98,6 +100,30 @@ static bool attrcpy(T& dest, const pugi::xml_node& src, const std::string& attr, return true; } +template +static bool nodecpy(T& dest, const pugi::xml_node& src, const std::string& node, bool opt = false) +{ + auto val = src.child(node.c_str()); + if(val == pugi::xml_node()) + { + if(!opt) + { + ERR(dgxmlparser, "Node %s not found in %s, offset %d\n", + node.data(), src.path().data(), (int)src.offset_debug()); + } + return opt; + } + + if(!assign(dest, val.text().as_string())) + { + ERR(dgxmlparser, "Attribute %s could not be assigned, offset %d\n", + node.data(), (int)src.offset_debug()); + return false; + } + + return true; +} + bool parseDrumkitFile(const std::string& filename, DrumkitDOM& dom) { bool res = true; @@ -112,16 +138,50 @@ bool parseDrumkitFile(const std::string& filename, DrumkitDOM& dom) return false; } - //TODO: handle xml version - pugi::xml_node drumkit = doc.child("drumkit"); - res &= attrcpy(dom.name, drumkit, "name"); + dom.version = "1.0"; res &= attrcpy(dom.version, drumkit, "version", true); - res &= attrcpy(dom.description, drumkit, "description"); dom.samplerate = 44100.0; res &= attrcpy(dom.samplerate, drumkit, "samplerate", true); + // Use the old name and description attributes on the drumkit node as fallback + res &= attrcpy(dom.metadata.title, drumkit, "name", true); + res &= attrcpy(dom.metadata.description, drumkit, "description", true); + + pugi::xml_node metadata = drumkit.child("metadata"); + if(metadata != pugi::xml_node()) + { + auto& meta = dom.metadata; + res &= nodecpy(meta.version, metadata, "version", true); + res &= nodecpy(meta.title, metadata, "title", true); + pugi::xml_node logo = metadata.child("logo"); + if(logo != pugi::xml_node()) + { + res &= attrcpy(meta.logo, logo, "src", true); + } + res &= nodecpy(meta.description, metadata, "description", true); + res &= nodecpy(meta.license, metadata, "license", true); + res &= nodecpy(meta.notes, metadata, "notes", true); + res &= nodecpy(meta.author, metadata, "author", true); + res &= nodecpy(meta.email, metadata, "email", true); + res &= nodecpy(meta.website, metadata, "website", true); + pugi::xml_node image = metadata.child("image"); + if(image != pugi::xml_node()) + { + res &= attrcpy(meta.image, image, "src", true); + res &= attrcpy(meta.image_map, image, "map", true); + for(auto clickmap : image.children("clickmap")) + { + meta.clickmaps.emplace_back(); + res &= attrcpy(meta.clickmaps.back().instrument, + clickmap, "instrument", true); + res &= attrcpy(meta.clickmaps.back().colour, + clickmap, "colour", true); + } + } + } + pugi::xml_node channels = doc.child("drumkit").child("channels"); for(pugi::xml_node channel: channels.children("channel")) { @@ -212,7 +272,7 @@ bool parseInstrumentFile(const std::string& filename, InstrumentDOM& dom) } // Velocity groups are only part of v1.0 instruments. - if(dom.version == "1.0") + if(dom.version == "1" || dom.version == "1.0" || dom.version == "1.0.0") { pugi::xml_node velocities = instrument.child("velocities"); for(pugi::xml_node velocity: velocities.children("velocity")) @@ -221,13 +281,12 @@ bool parseInstrumentFile(const std::string& filename, InstrumentDOM& dom) res &= attrcpy(dom.velocities.back().lower, velocity, "lower"); res &= attrcpy(dom.velocities.back().upper, velocity, "upper"); - for(pugi::xml_node sampleref: velocity.children("sampleref")) + for(auto sampleref : velocity.children("sampleref")) { dom.velocities.back().samplerefs.emplace_back(); - res &= attrcpy(dom.velocities.back().samplerefs.back().probability, - sampleref, "probability"); - res &= attrcpy(dom.velocities.back().samplerefs.back().name, - sampleref, "name"); + auto& sref = dom.velocities.back().samplerefs.back(); + res &= attrcpy(sref.probability, sampleref, "probability"); + res &= attrcpy(sref.name, sampleref, "name"); } } } diff --git a/src/domloader.cc b/src/domloader.cc index 8a133fa..ad97165 100644 --- a/src/domloader.cc +++ b/src/domloader.cc @@ -56,9 +56,9 @@ bool DOMLoader::loadDom(const std::string& basepath, { settings.has_bleed_control.store(false); - drumkit._name = dom.name; + drumkit._name = dom.metadata.title; drumkit._version = dom.version; - drumkit._description = dom.description; + drumkit._description = dom.metadata.description; drumkit._samplerate = dom.samplerate; for(const auto& channel: dom.channels) diff --git a/src/instrument.h b/src/instrument.h index 6e59681..125d7ab 100644 --- a/src/instrument.h +++ b/src/instrument.h @@ -67,15 +67,11 @@ public: float getMinPower() const; private: - friend class DOMLoader; + // For unit-tests: friend class DOMLoaderTest; // For parser: - friend class InstrumentParser; - friend class DrumKitParser; - - // For unit-tests: - friend class InstrumentParserTest; + friend class DOMLoader; void* magic; diff --git a/src/rangemap.h b/src/rangemap.h index e53cbe8..6448e0e 100644 --- a/src/rangemap.h +++ b/src/rangemap.h @@ -37,7 +37,7 @@ public: std::vector get(T1 at); private: - friend class InstrumentParserTest; + friend class DOMLoaderTest; std::multimap, T2> values; }; diff --git a/src/sample.h b/src/sample.h index 48f94ac..223648f 100644 --- a/src/sample.h +++ b/src/sample.h @@ -47,9 +47,7 @@ public: private: friend class DOMLoader; friend class DOMLoaderTest; - friend class InstrumentParser; friend class PowerList; - friend class InstrumentParserTest; void addAudioFile(InstrumentChannel* instrument_channel, AudioFile* audio_file); -- cgit v1.2.3