Follow-up Comment #7, patch #3826 (project freeciv):

I don't think there's any actual need for double removal, but otherwise
destroy_base() would need to either be passed boolean parameter telling if
caller is removing base itself, or it would need to be able to deduct the need
itself. Some callers other than tile_change_terrain() need it.
Or maybe tile_change_terrain() could remove base only if callback is not set,
and to rely on *any* callback ever assigned there to take care of the removal.

    _______________________________________________________

Reply to this item at:

  <http://gna.org/patch/?3826>

_______________________________________________
  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