Fix Armament not working properly with value 0 in BurstDelays

[PunkPun]

Supersedes: #20601

This allows us to simplify yaml definitions, as I've done in all non-d2k mods.
It will also make it easier for the encyclopedia to read armament definitions when we get to it.

[dnqbob]

This allows us to simplify yaml definitions, as I've done in all non-d2k mods.
It will also make it easier for the encyclopedia to read armament definitions when we get to it.

Add: by using this properly instead of massing armaments we can also get performance improve, since the armaments are traversed every time when AutoTarget check target and Attackbase fire at target.

[michaeldgg2]

I tested this change on OpenE2140 and it works fine. Here are the weapon and armament definition:

ucs_vehicles_tiger_assault:
	ReloadDelay: 40
	Report: 5.smp
	ValidTargets: Ground
	Burst: 8
	BurstDelays: 0, 10, 0, 10, 0, 10, 0

Armament@PRIMARY:
	Weapon: ucs_vehicles_tiger_assault
	Recoil: 64
	RecoilRecovery: 38
	LocalOffset: 100,-410,0, 100,410,0
	LocalYaw: 0, 0

It does not look as concise as the BurstPerFire (from superseded PR), but it does the job.

[Mailaender]

Changelog