Changeset 23 for cpp/f8-to-f1/conv_f8tof1.cpp

Timestamp:

06/27/09 11:12:12 (15 years ago)

Author:

mwajcht

Message:

Bug-fix in erasing elements from vectors in 'for' loops - use integer loop counters instead of iterators (after last element of a vector is erased incremented iterator does not point to vector.end())

File:

• cpp/f8-to-f1/conv_f8tof1.cpp (modified) (1 diff)

cpp/f8-to-f1/conv_f8tof1.cpp

@@ -702 +702 @@
                          subProdIter != prodIter->second->subproductions.end(); subProdIter++) {
                         SubProduction subProduction = *subProdIter;
-                        for (vector<ActionStrP>::iterator actionIter = subProduction.actions.begin();
-                                 actionIter != subProduction.actions.end(); actionIter++) {
-                                if ((*actionIter).action == NULL) {
-                                        subProduction.actions.erase(actionIter);
+                        for (int i = 0; i < subProduction.actions.size(); i++) {
+                                if (subProduction.actions.at(i).action == NULL) {
+                                        subProduction.actions.erase(subProduction.actions.begin() + i);
                                 }
                         }