Code review comment for lp:~widelands-dev/widelands/amazons-coding-changes

Benedikt Straub (nordfriese) wrote :

The barbarian sites modified in this branch are only testcases, not intended for merging.
The new amazons tribe (under development in another branch) will have a "Gardening Center" whose only purpose is that the worker walks around and turns the terrain nearby into a different, more fertile terrain type.

The sentry idea is that this "Treetop Sentry" is a platform in the branches of a tree. Internally, I have created a value "built_over_immovable" in addition to the Size value – a building with this value set can be built only on a player-owned node where the max nodecaps is >= the building´s size and an immovable with the required attribute (in this case: big tree) stands on the node. The immovable is then removed when placing the building, but I hacked the animations so it looks as if it were still there.

The AI is currently unable to build this sentry, and it won´t build a gardening center either, but otherwise it should be unaffected.

« Back to merge proposal