[Freeciv-Dev] Re: PATCH: AI cleanup Version 2

[Gregory Berkolaiko <gberkolaiko@xxxxxxxxxxx>]

I looked at your patch.

It's absolutely fine apart from the comment to unit_move_turns
You say: "Things like railroad, roads will make the actual move cost
lower than the given estimation. Mountains, swamps will increase actual
move cost"
But the estimation is based on the warmap which takes into account swamps
and roads.  I suggest you remove the last two sentences.  Instead add:
"VERY IMPORTANT: the function assumes that there is ready warmap for
punit at (x, y)"

Maybe you should just send in the patch with the function unit_move_turns
and then, separately, all other changes.  By me the current patch is
fine. But Raimar might want to split it up ;)

And to make it 100% safe run autogame and compare the saves.

Best,
G.


__________________________________________________
Do You Yahoo!?
Everything you'll ever need on one web page from News and Sport to Email and 
Music Charts
http://uk.my.yahoo.com