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