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.

1 files changed, 2 insertions, 2 deletions

diff --git a/source/blender/collada/collada_utils.cpp b/source/blender/collada/collada_utils.cpp
index f740fcee664..dd5611c4bef 100644
--- a/source/blender/collada/collada_utils.cpp
+++ b/source/blender/collada/collada_utils.cpp
@@ -525,7 +525,7 @@ BoneExtensionMap &BoneExtensionManager::getExtensionMap(bArmature *armature)
 BoneExtensionManager::~BoneExtensionManager()
 {
   std::map<std::string, BoneExtensionMap *>::iterator map_it;
-  for (map_it = extended_bone_maps.begin(); map_it != extended_bone_maps.end(); map_it++) {
+  for (map_it = extended_bone_maps.begin(); map_it != extended_bone_maps.end(); ++map_it) {
     BoneExtensionMap *extended_bones = map_it->second;
     for (BoneExtensionMap::iterator ext_it = extended_bones->begin();
          ext_it != extended_bones->end();
@@ -916,7 +916,7 @@ bool bc_is_animated(BCMatrixSampleMap &values)
 
   BCMatrixSampleMap::iterator it;
   const BCMatrix *refmat = NULL;
-  for (it = values.begin(); it != values.end(); it++) {
+  for (it = values.begin(); it != values.end(); ++it) {
     const BCMatrix *matrix = it->second;
 
     if (refmat == NULL) {