Missile / Super Missile deals incorrect damage about nxengine-evo

[isage]

Ugh. So, in original game on contact each missile spawns another invisible "bullet" with a predefined lifetime for an each level:
Missile: 10/15/5
Super Missile: 10/14/6
Each frame this "bullet" deals damage to an npc: 1 for missile, and 2 for super missile. Each 3 frames it spawns explosion effect and 2 smoke clouds.
This works fine for lv1 and lv2, however explosions from lv3 missiles for some reason sometime live for another frame, dealing additional dmg.
When shot from a large distance there's large pauses between missile hits and so only one (last) missile deals damage for additional frame. When shot from small distance - two last missiles deal damage for additional frame.
This actually looks like some bug in bullet handling in original game, because there's absolutely no special logic for that in code.

[isage]

Oh god, i found it. First of all, frame times above are wrong, they all should be +1 (or not, depending on Pixel's intention)
So: there's an array of bullets. SetBullet adds new bullet to the first found empty slot. ActBullet and HitNpCharBullet just go through this array.
Now, in missile ai calls SetBullet to spawn explosion, that deals actual damage.
Because it's called inside ActBullet loop, if explosion is added to the end of bullet array - it's respective ai gets called on the same frame, thus not dealing damage on this frame.
However, when explosion is added to array somewhere before missile, that spawned it - oops, it's ai is not called on that frame, but only on next one (and on next one it already deals damage, because HitNpCharBullet goes before ActBullet).

Ugh.

[isage]

Now, the only problem is i need to somehow reproduce this bug, without completely rewriting how bullets work (they are just objects, without some additional queue)

[isage]

Fixed in 8dab4de
Note that sprites.sif is modified.