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/BCAnimationSampler.h | |
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/BCAnimationSampler.h')
-rw-r--r-- | source/blender/collada/BCAnimationSampler.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/source/blender/collada/BCAnimationSampler.h b/source/blender/collada/BCAnimationSampler.h index 3d156134260..e8d2ab56ae7 100644 --- a/source/blender/collada/BCAnimationSampler.h +++ b/source/blender/collada/BCAnimationSampler.h @@ -47,7 +47,7 @@ class BCAnimation { ~BCAnimation() { BCAnimationCurveMap::iterator it; - for (it = curve_map.begin(); it != curve_map.end(); it++) { + for (it = curve_map.begin(); it != curve_map.end(); ++it) { delete it->second; } @@ -79,7 +79,7 @@ class BCSampleFrame { ~BCSampleFrame() { BCSampleMap::iterator it; - for (it = sampleMap.begin(); it != sampleMap.end(); it++) { + for (it = sampleMap.begin(); it != sampleMap.end(); ++it) { BCSample *sample = it->second; delete sample; } |