Changes in src/Actions/UndoRedoHelpers.cpp [596cfa:917300]

File:

• src/Actions/UndoRedoHelpers.cpp (modified) (1 diff)

src/Actions/UndoRedoHelpers.cpp

@@ -131 +131 @@
 {
   bool status = true;
-  for(std::vector< BondInfo >::const_iterator iter = bonds.begin();
-      iter != bonds.end(); ++iter)
-    if (!(*iter).RecreateBond())
+  std::vector< BondInfo >::const_iterator iter = bonds.begin();
+  for(;iter != bonds.end(); ++iter) {
+    if (!(*iter).RecreateBond()) {
       status = false;
+      break;
+    }
+  }
+  if (!status) {
+    // remove all added bonds again
+    for(std::vector< BondInfo >::const_iterator removeiter = bonds.begin();
+        removeiter != iter; ++removeiter) {
+      removeiter->RemoveBond();
+    }
+  }
   return status;
 }