We CURRENTLY have a new bug (not a regression but a new regression test) - where we get helgrind failures.
Discussed with KDJ/Sterling
I think the plan is to make an OPTIONAL param to containers like Sequnece_Array - if iterators are autosafe-patching.
THEN - its the responsabity of the callers to assure iterator acessed syncrhonzed with the owning container (patch but not lcoking other than debug::assertexternalylsynconized)
But most of the stroika code - loses the patching logic. So runs FASTER (and more bug free)
Maybe include in DEBUG mode maintaining the linked list (or something similar) to DETECT bugs and assert out. Maybe on iteartor have a CHANGE COUNT. And then to assert in iterator that they cannot be used after a change to the owning container!
Conssider i = s.begin();
i++; // assert ciz s 'changed'.
ALSO - make Remove() method on Sequnce (and maybe others) return an Iterator to next item.