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

Reply via email to