Fix error found by cppcheck - modifying a container invalidates its iterators,
so erasing from a list and then continuing to access the iterator over that list is a real nono. The standard idiom in stl parlance is cont.erase(remove_if())
In this case, to apply that idiom means splitting this into a futher functor.