diff options
| author | David Robillard <d@drobilla.net> | 2010-02-02 20:37:50 +0000 |
|---|---|---|
| committer | David Robillard <d@drobilla.net> | 2010-02-02 20:37:50 +0000 |
| commit | a645d2b8be4d7d31f6eef1649156b166a01e0c31 (patch) | |
| tree | 3d83c08e9a5460cc6582482bcfc673dff956a1e7 /src/client/NodeModel.cpp | |
| parent | 653679e967f11a0e008521885fcaf6994d2961fa (diff) | |
| download | ingen-a645d2b8be4d7d31f6eef1649156b166a01e0c31.tar.gz ingen-a645d2b8be4d7d31f6eef1649156b166a01e0c31.tar.bz2 ingen-a645d2b8be4d7d31f6eef1649156b166a01e0c31.zip | |
Use Glib string interning (quarks) to make Path/URI operator== very fast.
This avoids a ton of string comparison overhead in Ingen when setting various
properties (e.g. "ingen:value" was compared several times every time a port
value was changed, now this is just a single pointer comparison and the full
round trip of a value change does no string comparison at all, but is still
property based and RDFey).
git-svn-id: http://svn.drobilla.net/lad/trunk/ingen@2408 a436a847-0d15-0410-975c-d299462d15a1
Diffstat (limited to 'src/client/NodeModel.cpp')
| -rw-r--r-- | src/client/NodeModel.cpp | 9 |
1 files changed, 4 insertions, 5 deletions
diff --git a/src/client/NodeModel.cpp b/src/client/NodeModel.cpp index ea419e3a..df72c9e2 100644 --- a/src/client/NodeModel.cpp +++ b/src/client/NodeModel.cpp @@ -119,7 +119,7 @@ NodeModel::add_child(SharedPtr<ObjectModel> c) bool NodeModel::remove_child(SharedPtr<ObjectModel> c) { - assert(c->path().is_child_of(_path)); + assert(c->path().is_child_of(path())); assert(c->parent().get() == this); //bool ret = ObjectModel::remove_child(c); @@ -137,7 +137,7 @@ void NodeModel::add_port(SharedPtr<PortModel> pm) { assert(pm); - assert(pm->path().is_child_of(_path)); + assert(pm->path().is_child_of(path())); assert(pm->parent().get() == this); Ports::iterator existing = find(_ports.begin(), _ports.end(), pm); @@ -151,11 +151,10 @@ NodeModel::add_port(SharedPtr<PortModel> pm) SharedPtr<PortModel> -NodeModel::get_port(const string& port_name) const +NodeModel::get_port(const Raul::Symbol& symbol) const { - assert(port_name.find("/") == string::npos); for (Ports::const_iterator i = _ports.begin(); i != _ports.end(); ++i) - if ((*i)->path().name() == port_name) + if ((*i)->symbol() == symbol) return (*i); return SharedPtr<PortModel>(); } |