Summary: Ai settler cityspot reservation assumes citymindist
                 Project: Freeciv
            Submitted by: cazfi
            Submitted on: Wed 16 Feb 2011 02:07:45 AM EET
                Category: ai
                Severity: 3 - Normal
                Priority: 5 - Normal
                  Status: Ready For Test
             Assigned to: None
        Originator Email: 
             Open/Closed: Open
         Discussion Lock: Any
        Operating System: None
         Planned Release: 2.2.5, 2.3.0



Ai settler cityspot reservation code assumes that minimum distance is one
given by ruleset option min_dist_bw_cities. Actually server setting
citymindist overrides it.

Fix attached.

For TRUNK I prefer patch #2537 which completely removes min_dist_bw_cities.
We have not that often fixed ai performance bugs in stable branches. This one
should clearly go to S2_3 which is still stabilizing. S2_2 is more open for
discussion. I think this bug degrades ai performance so much when citymindist
has been set, and fix is so low risk, that it is worth including to S2_2.


File Attachments:

Date: Wed 16 Feb 2011 02:07:45 AM EET  Name: AiSMinDist.diff  Size: 790B  
By: cazfi



Reply to this item at:


  Message sent via/by Gna!

Freeciv-dev mailing list

Reply via email to