This patch removes armour lines from unit templates that are unnecessaryDifferent soldier types have different attacks. This patch therefore moves the attack sound groups to the template where the attack damage is actually defined, rather than having it in their grandparents. This is already the case for many (e.g. javelineers, because:spearmen).
* they're already inherited from parents;
* all children have different values;
* only one child uses them.
Fewer duplication means things are easier to maintain (or tweak) in the long run.
See also D2744, D2850Also moved the <attack_slaughter> sound from `template_unit.xml` to only its three children that have <Attack/Slaughter>.