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:
Message sent via/by Gna!
Freeciv-dev mailing list