bulbazord accepted this revision. bulbazord added a comment. I like this. :)
================ Comment at: lldb/source/Utility/Broadcaster.cpp:392 } - m_event_map.erase(iter); + iter = m_event_map.erase(iter); } ---------------- I don't think you need to actually capture the iterator here? std::map::erase doesn't invalidate existing iterators* (so `begin` and `end` are fine) and we redefine `iter` at the beginning of this loop. *:https://en.cppreference.com/w/cpp/container/map/erase Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D150219/new/ https://reviews.llvm.org/D150219 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits