The change you did its enough.
Because CurWeapon is called whenever the HUD clip value changes (shooting and after reloading),
when using snipers scope, and when changing weapons (which is the goal in your code).
About Ham_Killed, basically you will eliminate the need of using a native to retrieve the killer.
And about FM_GetGameDescription, in our tests it's always called 1 time per second.
But there's something else:
PHP Code:
https://www.amxmodx.org/api/fakemeta_const
// This also gets called when the server is queried for information (for example, by a server browser tool)
FM_GetGameDescription,
Do you plan on adding more features on it?
We are optimizing Aim Realism code for months and We have got something that could be used in your plugin.
It's a code which is able to get all the weapons the players are currently holding with minimal CPU usage.
For example:
PHP Code:
AK47[3] // 3 Players are currently holding AK47
M4A1[5] // 5 Players are currently holding M4A1
We thought about creating a plugin which shows a weapon sprite in the left side of the screen instead of text and the number in front of it indicating how many players are holding it.
It only needs to use Ham_Item_Deploy, Ham_Killed, and FM_ClientDisconnect to update the status.
We will release Aim Realism unfinished soon.
We need to conclude a fail-less anti-cheat system which is able to catch any kind of Auto aim / Aimbot in less than 0.02s with minimal CPU usage.
But in case you are interested, We can share with you the code for getting current player weapons.