Follow-up Comment #1, patch #1754 (project freeciv):
I support this change!
Some time ago I started to look into the AI code and also tried to merge
aisettler.(c|h) and settler.(c|h). There should be rules defining which code
should be in server/advisors or ai (in doc/CodingStyle?). Most of the
ai/adv*.(c|h) files could be moved into server/advisors after they are
checked for code only needed by the AI.
Should amortize() go into a special server/advisors/advtools.(c|h) file? It
is needed all over the ai/advisor code.
The move of server/settler.(c|h) to server/advisor/autosettler.(c|h) is
missing in the patch file. The define lines are change for the file but it is
not moved.
_______________________________________________________
Reply to this item at:
<http://gna.org/patch/?1754>
_______________________________________________
Nachricht geschickt von/durch Gna!
http://gna.org/
_______________________________________________
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev
