MakeInfantry logic on BombClass resulting on `Neutral` side infantry BugFix

[Otamaa]

close #10

• Fix the issue when BombClass giving nullptr Techno owner resulting on Some Warhead affect no applied properly .
• Added Ownership for BombClass explosion anim to fix problem with CreateUnit and MakeInf .

[github-actions[bot]]

Nightly build for this pull request:

• compiled-dll-cacdba4c948a6d351fbf61e89db1bc9274a4ab46.zip This comment is automatic and is meant to allow guests to get latest nightly builds without registering. It is updated on every successful build.

[Otamaa]

Could you explain what did you do to fix it? Looks unconventional and bit hard to read. Edited it , now it look more simple and hopefully more readable !

[deathreaperz]

Maybe a late review, I can confirm that "Added Ownership for BombClass explosion anim to fix problem with CreateUnit and MakeInf" works.

[Otamaa]

Could you explain what did you do to fix it? Looks unconventional and bit hard to read.

Basically redirect the MapClass::DamageArea call to our dll and add an ownership from the BombClass itself this kind a hard way to do it because ares seems manipulating the Push stack there .

[Starkku]

Tested, appears to work. Also fixes another related issue where the vanilla code would attempt to create AnimClass even if AnimTypeClass is nullptr.

Took a liberty of fixing issue preventing the code from building as well as couple of other things and pushing it.