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:


  Nachricht geschickt von/durch Gna!

Freeciv-dev mailing list

Reply via email to