The main example I can think of is when you know there will be enemies after the location reload.Yes the problem occured when reloading while in fightmode. So what could happen?
May happen in multi-location dungeons and also some quest cases, for example.
Could be annoying for players if they have to grab their weapon again every time.
Also, what is the "LAi_SetPlayerType" line for? That doesn't affect "fight mode",
but it may potentially influence quests where a reload occurs while the player is supposed to remain ActorType.
So much of that old code is spread out everywhere; amazingly annoying!Maximus wrote that code a long time ago and it's spread to 4 places (Lai_fightparams too). One of them I fixed years ago but missed the others. So this could never have worked properly (until now). I say just leave it. I have tested it and it works very well.

What does LAi_GunSetUnload(chr) do? Just unloads your gun if you don't have the ammo for it?
Probably something with the itemsbox.c interface ALWAYS calling a "re-equip" function of some kind.I agree. This "cheat" is no good and nothing I have ever wanted. Don't know where to fix it though.