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

Reply via email to