This looks fine to me... A couple of minor points:<div><br></div><div>- Can you add some comments describing the iterator invalidation issues that motivated this design?</div><div><br></div><div>- I think the unit test needs the boiler plate comment block at the top.</div>
<div><br></div><div>- I'd name the test itself after the PR number rather than putting it in a comment (which is likely to not stay attached).<br></div>