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:


  Message sent via/by Gna!

Freeciv-dev mailing list

Reply via email to