[PartialCloning]

Originally Posted by wickedd That's true Bugsy but he checked if the weapon != knife, so anything after that will not be called. The reason I said is_user_connected is needless in this case is because he's not doing anything to the attacker. I may be wrong but I don't see any need to check in this particular situation. Now if he wouldn't have checked for the weapon then I would agree. P.S. If missed something please let me know.

I'm not sure why you're not directly pointing out why exactly he doesn't need to check the killer is connected. But I went ahead and looked into it, and for anyone wondering, it's because get_user_weapon returns 0 if the client isn't connected. So does get_user_origin.

Originally Posted by Hedgehog95 Can someone explain this to me?

get_user_weapon will return true if the kill is a nade kill and the user has their knife out at the time of the kill.

[klippy]

About Hedgehog's code and is_user_connected() check on the attacker - he has to have it because attacker can be any entity, not just a player. is_user_connected() checks if the entity is a valid connected player.

@PartialCloning, get_user_origin won't just return 0 on invalid player index, it will throw an error and halt the running function. Take a look here. You don't really want that to happen.

[PartialCloning]

Yeah for some reason I assumed he was checking if entity is within maxclients.