Crashing ("Debug assertion failed! Expression: vector subscript out of range") when removing colony with population of 0. about antsimulator

[marcoperic]

In simulation's update() function, I have a check at the very bottom (below checkForFights and updateColoniesStats) that checks to see if any colonies are extinct. It looks like this:
for (Colony& colony : colonies) { if (colony.eliminated) removeColony(colony.id); }

eliminated is a bool that is triggered when the ant population is equal to zero.

When debugging, I see that the error originates from removeColony(uint8_t colony_id) and occurs in the for loop with c.stopFightsWith(colony_id);`

From there, the next error takes place in the Colony class in the stopFightsWith function, inside of the for loop, at if(a.target) {
Where a.target seems to be the problem. Here is the call stack:

Note that the first four lines have line numbers that should match with yours. Everything below operator bool() Line 359 probably differs from your code.

[marcoperic]

My thoughts:
I don't think anything I added to your code managed to cause this problem.
I think the problem comes from trying to iterate over the Ants of a colony that no longer exists (was deleted because population was 0). Maybe there is a problem with Ant.target when the colony is extinct, when there are no ants, or when the colony no longer exists?

In any case, debugging keeps pointing to lines of code involving if (ants.target), even when I try to patch this bug.

Let me know if there is anything I can do to help!

Regards,
Marco

[johnBuffer]

Hello,
Thank you for your detailed analysis, I will look into this