Code review comment for lp:~ad-simplypeachy/widelands/ai-hints

Revision history for this message
Nasenbaer (nasenbaer) wrote :

I am a bit split towards this change - I have not checked the effect of this patch, however the idea I had when implementing the "is_basic" feature was, to force the defaultAI to build some build resource producing buildings faster than al the others - so e.g. wood hardener, as blackwood is very important for building up other buildings.

Therefor I am almost against this fix - the right way imho would be to fix/improve the "decission function" of the defaultAI where it is decided what buildings should be build next.

Maybe the positive effect of this branch could be achieved with carlis AI fixes... but I haven't tested it, yet.

review: Needs Information

« Back to merge proposal