diff --git a/src/App/Transactions.cpp b/src/App/Transactions.cpp
index ef4ac203a8..f61b97be94 100644
--- a/src/App/Transactions.cpp
+++ b/src/App/Transactions.cpp
@@ -234,13 +234,16 @@ void TransactionObject::applyChn(Document & /*Doc*/, DocumentObject * /*pcObj*/,
 {
     if (status == New || status == Chn) {
         // apply changes if any
-        if(!Forward){
+        if (!Forward) {
             std::map<const Property*,Property*>::const_reverse_iterator It;
-            for (It=_PropChangeMap.rbegin();It!=_PropChangeMap.rend();++It)
+            std::map<const Property*,Property*>::const_reverse_iterator rendIt = _PropChangeMap.rend();
+            for (It = _PropChangeMap.rbegin(); It != rendIt; ++It)
                 const_cast<Property*>(It->first)->Paste(*(It->second));
-        }else{
+        }
+        else {
             std::map<const Property*,Property*>::const_iterator It;
-            for (It=_PropChangeMap.begin();It!=_PropChangeMap.end();++It)
+            std::map<const Property*,Property*>::const_iterator endIt = _PropChangeMap.end();
+            for (It = _PropChangeMap.begin(); It != endIt; ++It)
                 const_cast<Property*>(It->first)->Paste(*(It->second));
         }
     }