diff options
| author | Sergey Sharybin <sergey.vfx@gmail.com> | 2019-02-18 18:24:51 +0300 |
|---|---|---|
| committer | Sergey Sharybin <sergey.vfx@gmail.com> | 2019-02-18 18:24:51 +0300 |
| commit | d02ad52b2de250fa371f664f1f2d08329e0b5a20 (patch) | |
| tree | d6ca23e8f739d8128c50a8040612ede912bcdd42 /source/blender/depsgraph/intern/node/deg_node_component.cc | |
| parent | c3ceefb6715bc17fd3a6a0f409eb27d50695681b (diff) | |
Depsgraph: Make node an owner of its name
The initial idea of using char pointer was to save some
memory since the dependency graph was kind of the one
with the main database.
Nowadays dependency graph should be separatable from the
main database and being self-sustainable.
Other issue which was caused by this pointer is the
re-tagging of operations during relations update: it is
possible to have node which as tagged for update but had
the owner of the name removed (i.e. driver or bone was
removed).
Diffstat (limited to 'source/blender/depsgraph/intern/node/deg_node_component.cc')
| -rw-r--r-- | source/blender/depsgraph/intern/node/deg_node_component.cc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/source/blender/depsgraph/intern/node/deg_node_component.cc b/source/blender/depsgraph/intern/node/deg_node_component.cc index 4d16ff7095c..06ede835a27 100644 --- a/source/blender/depsgraph/intern/node/deg_node_component.cc +++ b/source/blender/depsgraph/intern/node/deg_node_component.cc @@ -164,7 +164,7 @@ OperationNode *ComponentNode::find_operation(OperationIDKey key) const for (OperationNode *op_node : operations) { if (op_node->opcode == key.opcode && op_node->name_tag == key.name_tag && - STREQ(op_node->name, key.name)) + STREQ(op_node->name.c_str(), key.name)) { node = op_node; break; |