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/collada/BCSampleData.cpp | |
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/collada/BCSampleData.cpp')
-rw-r--r-- | source/blender/collada/BCSampleData.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/source/blender/collada/BCSampleData.cpp b/source/blender/collada/BCSampleData.cpp index 4ef13621c97..7e23a2de00f 100644 --- a/source/blender/collada/BCSampleData.cpp +++ b/source/blender/collada/BCSampleData.cpp @@ -23,7 +23,7 @@ BCSample::~BCSample() { BCBoneMatrixMap::iterator it; - for (it = bonemats.begin(); it != bonemats.end(); it++) { + for (it = bonemats.begin(); it != bonemats.end(); ++it) { delete it->second; } } @@ -50,7 +50,7 @@ const bool BCSample::get_value(std::string channel_target, const int array_index bname = bname.substr(2); bname = bc_string_before(bname, "\""); BCBoneMatrixMap::const_iterator it; - for (it = bonemats.begin(); it != bonemats.end(); it++) { + for (it = bonemats.begin(); it != bonemats.end(); ++it) { Bone *bone = it->first; if (bname == bone->name) { matrix = it->second; |