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
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev

Reply via email to