Weapon.CheckAmmo/DepleteAmmo bugs

[Talon1024]

Cross-post from the virtual CheckAmmo feature request thread:

The latest commit introduced some bugs.

In the WolfenDoom: Blade of Agony chapter 2 release (and the latest dev version), the MP40 and Gewehr 43 draw ammo from the ammo bank (Secondary ammo type) instead of the ammo magazine (Primary ammo type), but the Browning Auto-5 correctly draws ammo from the magazine instead of the bank.

The MP40 and Gewehr 43 can also continue firing with an empty magazine, but the Browning Auto-5 still works as expected.

EDIT: Looks like the Browning Auto-5 takes/gives ammo manually rather than letting the native code perform that function.

Trailblazer is also affected. However in that case, weapons with magazines do not draw ammo from either the magazine or the ammo bank when fired.

EDIT: Here is a modified version of the AK47 from Realm667 that demonstrates this bug. GZDoom builds before Jan 30 2018 do not have this bug.

[Talon1024]

Maybe it's just a typo? I noticed this line has a capital on altFire, which I'm guessing conflicts with this enum value.

EDIT: Yep. Fix is here.