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. 


> 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:


  Nachricht geschickt von/durch Gna!

Freeciv-dev mailing list

Reply via email to