Follow-up Comment #10, patch #3826 (project freeciv):
Oh, sure. These all assume that destroy_base() will call tile_remove_base().
I was mostly concerned about callbacks, because they could potentially *not*
call tile_remove_base(), which would require the extra call in
tile_change_terrain().
I'll open a different patch to drop the redundant call, and leave a comment
indicating that tile_change_terrain() relies on any callback set for
fc_funcs->destroy_base() to perform tile_remove_base().
_______________________________________________________
Reply to this item at:
<http://gna.org/patch/?3826>
_______________________________________________
Message sent via/by Gna!
http://gna.org/
_______________________________________________
Freeciv-dev mailing list
[email protected]
https://mail.gna.org/listinfo/freeciv-dev
