ZDoom

Makes sense that it would behave that way if it's being called twice.

If these functions are already checking for replacement before they call Spawn, then they shouldn't do the same again.

kind = kind->GetReplacement(); //This is called first
....
actor = Spawn (kind, pos, ALLOW_REPLACE); //ALLOW_REPLACE ensures GetReplacement() is called again

(This has nothing to do with the other CheckReplacement() bug report)

Spawning something via ACS (or the old Thing_Spawn* functions, which can be line specials by themselves) completely ignores e.IsFinal being explicitly set to 'true'.

[attachment=0]checkrepfinal_acsspawn_test.pk3[/attachment]
In this test PK3, ZombieMan is replaced with ShotgunGuy via CheckReplacemet(), with e.IsFinal being true.
ShotgunGuy is replaced by a Cacodemon copy via the 'replaces' keyword.

In the test map, one zombieman is placed via map editor, it gets replaced with a shotgunner as it should be. There are three mapspots which a ACS script will also spawn zombiemen at. However, it is here where e.IsFinal is ignored and cacodemons spawn instead. The ACS script uses various spawning functions.

I think I've narrowed it down to these three: [url=https://github.com/coelckers/gzdoom/blob/23a4daac23ed864961b769d5d6fb0fe139beb802/src/p_things.cpp#L55]P_Thing_Spawn[/url], [url=https://github.com/coelckers/gzdoom/blob/23a4daac23ed864961b769d5d6fb0fe139beb802/src/p_things.cpp#L166]P_Thing_Projectile[/url] and [url=https://github.com/coelckers/gzdoom/blob/master/src/p_acs.cpp#L3922]DLevelScript::DoSpawn[/url]

In particular, each of them has this pattern:
[code]kind = kind->GetReplacement(); //This is called first
....
actor = Spawn (kind, pos, ALLOW_REPLACE); //ALLOW_REPLACE ensures GetReplacement() is called again
[/code]