Follow-up Comment #4, patch #1744 (project freeciv):

update to fix the patch wrt the other changes

> This affects only AI players, right? It should remain in
> structure clearly marked as AI stuff. I'm against polluting
> other structures with parts of AI stuff, especially with
> something that is only relevant to current AI implementation. 

OK

> This one makes sense. It's not part of AI, but tells if AI
> code should be used at all. Actually I think I already had
> patch to move it away from ai_data somewhere. 

In another update

> I'm a bit undecided about this. Should all implementations of
> modular AI to implement same skill levels, or can they define
> skill levels differently? 

A clear guideline is needed if this value is used in the client(s)

(file #9530)
    _______________________________________________________

Additional Item Attachment:

File name: 20100719-05-trunk-rename-struct-player-ai_data-to-ai_common.patch
Size:109 KB


    _______________________________________________________

Reply to this item at:

  <http://gna.org/patch/?1744>

_______________________________________________
  Nachricht geschickt von/durch Gna!
  http://gna.org/


_______________________________________________
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev

Reply via email to