c++ - How to call erase with a reverse iterator using a for loop

Question

Regarding the answer provided here: How to call erase with a reverse iterator

The following results in a segmentation fault (upon ++it) when compiled in g++ 4.8.4 with -std=c++11. Am I misunderstanding the answer?

  std::map<int,int> testmap;
  testmap[0] = 1;
  for(auto it=testmap.rbegin(); it!=testmap.rend(); ++it) {
    testmap.erase( std::next(it).base() );
  }

Answer 1

erase invalidates iterator, you have to reconstruct it from return of erase:

it = std::map<int,int>::reverse_iterator(testmap.erase( std::next(it).base() ));

or (c++11)

it = decltype(it){testmap.erase( std::next(it).base() )};

Demo.

For completeness, here is what the corrected loop from the original question looks like (notice that the iterator increment has been removed from the for(...):

for (auto rit = testmap.rbegin(); rit != testmap.rend(); /* empty */) {
    if (WE_WANT_TO_ERASE(rit)) {
        rit = decltype(rit){ testmap.erase(std::next(rit).base()) };
    } else {
        ++rit;
    }
}