← Back to team overview

widelands-dev team mailing list archive

Re: [Merge] lp:~widelands-dev/widelands/ferry into lp:widelands


Diff comments:

> === modified file 'src/ai/ai_help_structs.h'
> --- src/ai/ai_help_structs.h	2019-05-25 08:20:22 +0000
> +++ src/ai/ai_help_structs.h	2019-05-31 11:03:05 +0000
> @@ -78,6 +78,7 @@
>  	kUpgradeExtends,
>  	kLogRefiner,
>  	kIronMine,
> +	// TODO(Nordfriese): Someone should update the AI code to handle buildings that need waterways enabled

The Ferry Yard is enabled only on maps with waterways allowed. It should get some special handling code, like the shipyard.
And the barbarian and frisian weaving mill are now enabled if seafaring and/or ferries are allowed (previously if and only if seafaring is allowed). Not sure if the AI automatically adapts to that change (esp. on maps with ferries allowed and seafaring forbidden) or some tweak is needed.

>  	kNeedsSeafaring,
>  	kSupportingProducer,
>  	kNeedsBerry,

Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/ferry.