Code review comment for lp:~widelands-dev/widelands/mines-worldsavior

Revision history for this message
Toni Förster (stonerl) wrote :

I do have one question regarding the empire.

The armorsmithy produces 2 armor_helmet with the changes made in this branch, instead of one. The question I have is: why does the trainingcamp need helmets? The soldiers are already equipped with one.

The barbarians for example don't have a helmet when they start as a soldier they just need an ax and when they level up the receive new improved gear.

Why not go the same route with the empire?

Option 1:

Normal soldiers receive only a wooden spear but no helmet.
When they level up they receive their helmets.

Option 2:

Normal soldiers receive a wooden spear and helmet, but the trainingcamp doesn't need one.

I just noticed this while merging trunk and fixing merge conflicts.

« Back to merge proposal