On 3/6/2018 10:18 AM, Jon Turney wrote:
(I also note we have also have another 'erase an element from a vector while we are iterating over it' here, so that needs fixing, as well)
I think this one might be OK. If I'm not mistaken, pkg.versions.erase(i++) passes a copy of i to erase, and then increments i before erase() has done its work. But I'm no expert on this.