Summary: refactor first portion of ai_airlift into
                 Project: Freeciv
            Submitted by: nixeagle
            Submitted on: Thursday 09/09/2010 at 03:58
                Category: ai
                Priority: 1 - Later
                  Status: None
                 Privacy: Public
             Assigned to: None
        Originator Email: 
             Open/Closed: Open
         Discussion Lock: Any
         Planned Release: 



New function combines original two loops into one loop and returns the city
with the greatest need of an airlift. 

Resulting code does the same exact thing as before, but reading and
understanding what the code does is much easier for this one portion of the

-- Above is the commit summery I made to my local git repository.

A little more detail, this is my first ever patch to this project. I'm
interested in hacking on and improving the game ai portions. To start I
thought I'd start reading through the code and fixing up what I can while I
work on groking the code base.

Please note that this patch is not strongly tested. I've checked that it does
not obviously break the ai, but I'm not setup yet for proper regression tests.
I will be by this weekend.

I've tried my best to adhere to the coding standards. However I don't yet
have a project specific emacs configuration, so please poke me if something
is not quite right.

Finally while I know the coding style wants no variable initializations at
the same time as the declarations... I'm really not sure how that is to be
done in find_neediest_airlift_city, and I see some other areas disregard this
particular rule. I'm all ears though :).


File Attachments:

Date: Thursday 09/09/2010 at 03:58  Name:
0001-refactor-first-portion-of-ai_airlift-into-find_needi.patch  Size: 3kB  
By: nixeagle
simple refactor.


Reply to this item at:


  Message sent via/by Gna!

Freeciv-dev mailing list

Reply via email to