Summary: refactor first portion of ai_airlift into
Submitted by: nixeagle
Submitted on: Thursday 09/09/2010 at 03:58
Priority: 1 - Later
Assigned to: None
Discussion Lock: Any
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 :).
Date: Thursday 09/09/2010 at 03:58 Name:
0001-refactor-first-portion-of-ai_airlift-into-find_needi.patch Size: 3kB
Reply to this item at:
Message sent via/by Gna!
Freeciv-dev mailing list