Follow-up Comment #11, bug #20498 (project freeciv): Also, while this is a fine change, I can't actually see how unit_list_iterate could do the wrong thing here, where only the current unit can be removed. Perhaps I'm just being dim. Has this change been observed to remove a symptom?
Another opportunity to prove I'm asleep: comment #5 says: > 1) It's done in fc_assert() so won't do anything in release build I don't see anything in the committed change about moving functionality out of asserts. Has this been dealt with? I can't spot the original problem. (This wasn't bug #20519, was it? -- but fix committed before the comment was made, and caused no bug anyway.) _______________________________________________________ Reply to this item at: <http://gna.org/bugs/?20498> _______________________________________________ Message sent via/by Gna! http://gna.org/ _______________________________________________ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev