diff options
author | Campbell Barton <ideasman42@gmail.com> | 2019-09-07 20:31:49 +0300 |
---|---|---|
committer | Campbell Barton <ideasman42@gmail.com> | 2019-09-07 21:08:10 +0300 |
commit | fd05f01be60128e70b7e53d566627c9e84b6a98d (patch) | |
tree | c49680b53d0a689e7a8766674e0bec3bc1207908 /source/blender/depsgraph/intern/builder/deg_builder_transitive.cc | |
parent | abeab4fcad6635e6932f5719918338414a20c726 (diff) |
Partially revert "Cleanup: use post increment/decrement"
This partially reverts commit 0b2d1badecc48b5cbff5ec088b29c6e9acc5e1d0
Post increment can deep-copy for C++ iterators, while in my own checks
GCC was able to optimize this to get the same output,
better follow C++ best practice and use pre-increment for iterators.
Diffstat (limited to 'source/blender/depsgraph/intern/builder/deg_builder_transitive.cc')
-rw-r--r-- | source/blender/depsgraph/intern/builder/deg_builder_transitive.cc | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/source/blender/depsgraph/intern/builder/deg_builder_transitive.cc b/source/blender/depsgraph/intern/builder/deg_builder_transitive.cc index 34d2afcfa66..13cf8e63832 100644 --- a/source/blender/depsgraph/intern/builder/deg_builder_transitive.cc +++ b/source/blender/depsgraph/intern/builder/deg_builder_transitive.cc @@ -92,7 +92,7 @@ void deg_graph_transitive_reduction(Depsgraph *graph) * set/cleared. */ /* TODO: there will be other types in future, so iterators above * need modifying. */ - it_rel++; + ++it_rel; } else if (rel->from->custom_flags & OP_REACHABLE) { rel->unlink(); @@ -100,7 +100,7 @@ void deg_graph_transitive_reduction(Depsgraph *graph) num_removed_relations++; } else { - it_rel++; + ++it_rel; } } } |