Code review comment for lp:~widelands-dev/widelands/ai-post-b19-2

Revision history for this message
GunChleoc (gunchleoc) wrote :

> Savegame compatibility is quite tricky. I will look at this but I believe that proper strategy is just to ignore previous AI data (from older packet) and initialize AI data from scratch.

I agree - we have to run the loading to eat uo the packets, but
initializing the AI data from scratch will be the way to go.

> I also dont like addressing buildings by name, and when we discuss AI changes on forum, most players gives advices specific for buildings&tribes. I am trying to avoid this as much as possible.
>
> I can introduce bunch of new ai_hints like "is_bakery", though opposite would be better (per each tribe):
> bakery:"barbarian_bakery"
> woodcutter:"barbarian_lumberjack"

You mean like we have for the port & barracks? I could implement that
pretty easily.

« Back to merge proposal