URL: <http://gna.org/patch/?1970>
Summary: Factor out destroy_base() Project: Freeciv Submitted by: jtn Submitted on: Wednesday 09/15/10 at 23:46 Category: None Priority: 5 - Normal Status: Ready For Test Privacy: Public Assigned to: jtn Originator Email: Open/Closed: Open Discussion Lock: Any Planned Release: _______________________________________________________ Details: Nearly identical code exists to remove bases in unit_pillage_base() and create_base() (to remove conflicting bases). The patch pulls it out into a new function destroy_base(). The create_base case had a call to map_claim_ownership(ptile, NULL, NULL) that the unit_pillage_base case didn't; after staring at the code for a bit, I've concluded that it shouldn't be necessary in either case, so it doesn't exist in the new function. _______________________________________________________ File Attachments: ------------------------------------------------------- Date: Wednesday 09/15/10 at 23:46 Name: trunk-destroy-base.diff Size: 4kB By: jtn trunk r18039: factor out destroy_base() <http://gna.org/patch/download.php?file_id=10366> _______________________________________________________ Reply to this item at: <http://gna.org/patch/?1970> _______________________________________________ Message sent via/by Gna! http://gna.org/ _______________________________________________ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev