Simplify GUIOBjectState a bit by just walking the XML
directly rather than maintaining internal data structures. git-svn-id: svn://localhost/ardour2/branches/3.0@11287 d708f5d6-7413-0410-9779-e7cbd77b26cf
This commit is contained in:
parent
d03d0363a4
commit
3396a9a851
@ -18,11 +18,8 @@
|
||||
*/
|
||||
|
||||
#include <iostream>
|
||||
#include <iomanip>
|
||||
#include <sstream>
|
||||
|
||||
#include <boost/variant/static_visitor.hpp>
|
||||
|
||||
#include "gui_object.h"
|
||||
#include "i18n.h"
|
||||
|
||||
@ -30,92 +27,67 @@ using std::string;
|
||||
|
||||
const string GUIObjectState::xml_node_name (X_("GUIObjectState"));
|
||||
|
||||
GUIObjectState::~GUIObjectState ()
|
||||
GUIObjectState::GUIObjectState ()
|
||||
: _state (X_("GUIObjectState"))
|
||||
{
|
||||
clear_maps ();
|
||||
|
||||
}
|
||||
|
||||
void
|
||||
GUIObjectState::clear_maps ()
|
||||
XMLNode *
|
||||
GUIObjectState::find_node (const string& id) const
|
||||
{
|
||||
_property_maps.clear ();
|
||||
XMLNodeList const & children = _state.children ();
|
||||
for (XMLNodeList::const_iterator i = children.begin(); i != children.end(); ++i) {
|
||||
if ((*i)->name() != X_("Object")) {
|
||||
continue;
|
||||
}
|
||||
|
||||
XMLProperty* p = (*i)->property (X_("id"));
|
||||
if (p && p->value() == id) {
|
||||
return *i;
|
||||
}
|
||||
}
|
||||
|
||||
return 0;
|
||||
}
|
||||
|
||||
class gos_string_vistor : public boost::static_visitor<> {
|
||||
public:
|
||||
gos_string_vistor (std::ostream& o)
|
||||
: stream (o) {}
|
||||
/** Get a string from our state.
|
||||
* @param id property of Object node to look for.
|
||||
* @param prop_name name of the Object property to return.
|
||||
* @param empty if non-0, filled in with true if the property is currently non-existant, otherwise false.
|
||||
* @return value of property `prop_name', or empty.
|
||||
*/
|
||||
|
||||
void operator() (const int64_t& i) {
|
||||
stream << i;
|
||||
}
|
||||
|
||||
void operator() (const std::string& s) {
|
||||
stream << s;
|
||||
}
|
||||
|
||||
private:
|
||||
std::ostream& stream;
|
||||
};
|
||||
|
||||
std::string
|
||||
GUIObjectState::get_string (const std::string& id, const std::string& prop_name, bool* empty)
|
||||
string
|
||||
GUIObjectState::get_string (const string& id, const string& prop_name, bool* empty)
|
||||
{
|
||||
StringPropertyMap::iterator i = _property_maps.find (id);
|
||||
|
||||
if (i == _property_maps.end()) {
|
||||
XMLNode* child = find_node (id);
|
||||
if (!child) {
|
||||
if (empty) {
|
||||
*empty = true;
|
||||
}
|
||||
return string();
|
||||
return string ();
|
||||
}
|
||||
|
||||
const PropertyMap& pmap (i->second);
|
||||
PropertyMap::const_iterator p = pmap.find (prop_name);
|
||||
|
||||
if (p == pmap.end()) {
|
||||
XMLProperty* p = child->property (prop_name);
|
||||
if (!p) {
|
||||
if (empty) {
|
||||
*empty = true;
|
||||
}
|
||||
return string();
|
||||
return string ();
|
||||
}
|
||||
|
||||
std::stringstream ss;
|
||||
gos_string_vistor gsv (ss);
|
||||
|
||||
boost::apply_visitor (gsv, p->second);
|
||||
|
||||
if (empty) {
|
||||
*empty = false;
|
||||
}
|
||||
|
||||
return ss.str ();
|
||||
return p->value ();
|
||||
}
|
||||
|
||||
XMLNode&
|
||||
GUIObjectState::get_state () const
|
||||
{
|
||||
XMLNode* root = new XMLNode (xml_node_name);
|
||||
|
||||
for (StringPropertyMap::const_iterator i = _property_maps.begin(); i != _property_maps.end(); ++i) {
|
||||
|
||||
const PropertyMap& pmap (i->second);
|
||||
XMLNode* id_node = new XMLNode (X_("Object"));
|
||||
|
||||
id_node->add_property ("id", i->first);
|
||||
|
||||
for (PropertyMap::const_iterator p = pmap.begin(); p != pmap.end(); ++p) {
|
||||
std::stringstream ss;
|
||||
gos_string_vistor gsv (ss);
|
||||
boost::apply_visitor (gsv, p->second);
|
||||
id_node->add_property (p->first.c_str(), ss.str());
|
||||
}
|
||||
|
||||
root->add_child_nocopy (*id_node);
|
||||
}
|
||||
|
||||
|
||||
return *root;
|
||||
return *new XMLNode (_state);
|
||||
}
|
||||
|
||||
int
|
||||
@ -125,37 +97,10 @@ GUIObjectState::set_state (const XMLNode& node)
|
||||
return -1;
|
||||
}
|
||||
|
||||
clear_maps ();
|
||||
|
||||
for (XMLNodeList::const_iterator i = node.children().begin(); i != node.children().end(); ++i) {
|
||||
if ((*i)->name() == X_("Object")) {
|
||||
|
||||
XMLNode* child = (*i);
|
||||
const XMLProperty* idp = child->property (X_("id"));
|
||||
|
||||
if (!idp) {
|
||||
continue;
|
||||
}
|
||||
|
||||
string id (idp->value());
|
||||
|
||||
for (XMLPropertyList::const_iterator p = child->properties().begin(); p != child->properties().end(); ++p) {
|
||||
/* note that this always sets the property with
|
||||
a string value, and so is not equivalent to
|
||||
a call made by the program that passed a
|
||||
scalar.
|
||||
*/
|
||||
if ((*p)->name() != X_("id")) {
|
||||
set (id, (*p)->name(), (*p)->value());
|
||||
}
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
_state = node;
|
||||
return 0;
|
||||
}
|
||||
|
||||
|
||||
void
|
||||
GUIObjectState::load (const XMLNode& node)
|
||||
{
|
||||
@ -165,22 +110,27 @@ GUIObjectState::load (const XMLNode& node)
|
||||
GUIObjectState&
|
||||
GUIObjectState::operator= (const GUIObjectState& other)
|
||||
{
|
||||
_property_maps = other._property_maps;
|
||||
|
||||
_state = other._state;
|
||||
return *this;
|
||||
}
|
||||
|
||||
/** @return begin iterator into our StringPropertyMap */
|
||||
GUIObjectState::StringPropertyMap::const_iterator
|
||||
GUIObjectState::begin () const
|
||||
std::list<string>
|
||||
GUIObjectState::all_ids () const
|
||||
{
|
||||
return _property_maps.begin ();
|
||||
std::list<string> ids;
|
||||
XMLNodeList const & children = _state.children ();
|
||||
for (XMLNodeList::const_iterator i = children.begin(); i != children.end(); ++i) {
|
||||
if ((*i)->name() != X_("Object")) {
|
||||
continue;
|
||||
}
|
||||
|
||||
XMLProperty* p = (*i)->property (X_("id"));
|
||||
if (p) {
|
||||
ids.push_back (p->value ());
|
||||
}
|
||||
}
|
||||
|
||||
return ids;
|
||||
}
|
||||
|
||||
/** @return end iterator into our StringPropertyMap */
|
||||
GUIObjectState::StringPropertyMap::const_iterator
|
||||
GUIObjectState::end () const
|
||||
{
|
||||
return _property_maps.end ();
|
||||
}
|
||||
|
||||
|
@ -28,18 +28,12 @@
|
||||
#include "pbd/xml++.h"
|
||||
#include "pbd/id.h"
|
||||
|
||||
class GUIObjectState {
|
||||
private:
|
||||
typedef boost::variant<int64_t,std::string> Variant;
|
||||
typedef std::map<std::string,Variant> PropertyMap;
|
||||
#include "i18n.h"
|
||||
|
||||
public:
|
||||
typedef std::map<std::string,PropertyMap> StringPropertyMap;
|
||||
|
||||
~GUIObjectState();
|
||||
|
||||
StringPropertyMap::const_iterator begin () const;
|
||||
StringPropertyMap::const_iterator end () const;
|
||||
class GUIObjectState
|
||||
{
|
||||
public:
|
||||
GUIObjectState ();
|
||||
|
||||
XMLNode& get_state () const;
|
||||
int set_state (const XMLNode&);
|
||||
@ -52,19 +46,24 @@ class GUIObjectState {
|
||||
std::string get_string (const std::string& id, const std::string& prop_name, bool* empty = 0);
|
||||
|
||||
template<typename T> void set (const std::string& id, const std::string& prop_name, const T& val) {
|
||||
StringPropertyMap::iterator i = _property_maps.find (id);
|
||||
|
||||
if (i != _property_maps.end()) {
|
||||
i->second[prop_name] = val;
|
||||
} else {
|
||||
_property_maps[id] = PropertyMap();
|
||||
_property_maps[id][prop_name] = val;
|
||||
XMLNode* child = find_node (id);
|
||||
if (!child) {
|
||||
child = new XMLNode (X_("Object"));
|
||||
child->add_property (X_("id"), id);
|
||||
_state.add_child_nocopy (*child);
|
||||
}
|
||||
}
|
||||
private:
|
||||
StringPropertyMap _property_maps;
|
||||
|
||||
void clear_maps ();
|
||||
std::stringstream s;
|
||||
s << val;
|
||||
child->add_property (prop_name.c_str(), s.str());
|
||||
}
|
||||
|
||||
std::list<std::string> all_ids () const;
|
||||
|
||||
private:
|
||||
XMLNode* find_node (const std::string &) const;
|
||||
|
||||
XMLNode _state;
|
||||
};
|
||||
|
||||
|
||||
|
@ -249,15 +249,15 @@ MidiTimeAxisView::set_route (boost::shared_ptr<Route> rt)
|
||||
* the children.
|
||||
*/
|
||||
|
||||
GUIObjectState& gui_state = gui_object_state ();
|
||||
for (GUIObjectState::StringPropertyMap::const_iterator i = gui_state.begin(); i != gui_state.end(); ++i) {
|
||||
list<string> gui_ids = gui_object_state().all_ids ();
|
||||
for (list<string>::const_iterator i = gui_ids.begin(); i != gui_ids.end(); ++i) {
|
||||
PBD::ID route_id;
|
||||
bool has_parameter;
|
||||
Evoral::Parameter parameter (0, 0, 0);
|
||||
|
||||
bool const p = AutomationTimeAxisView::parse_state_id (i->first, route_id, has_parameter, parameter);
|
||||
bool const p = AutomationTimeAxisView::parse_state_id (*i, route_id, has_parameter, parameter);
|
||||
if (p && route_id == _route->id () && has_parameter) {
|
||||
create_automation_child (parameter, string_is_affirmative (gui_object_state().get_string (i->first, X_("visible"))));
|
||||
create_automation_child (parameter, string_is_affirmative (gui_object_state().get_string (*i, X_("visible"))));
|
||||
}
|
||||
}
|
||||
}
|
||||
|
Loading…
Reference in New Issue
Block a user