[Mitchell]

Seems dumb, Client_HasWeapon works with a client index, why are you converting what ever variable from what it's defined as to int?

[klippy]

Originally Posted by Mitchell Hmm however it does return a boolean. Could old syntax do that without the right return type? https://github.com/bcserv/smlib/blob...ents.inc#L1358

I believe the SP compiler doesn't infer types from return statements. So that function does in fact return an int. That function should've been tagged with bool:.

Originally Posted by Mitchell Seems dumb, Client_HasWeapon works with a client index, why are you converting what ever variable from what it's defined as to int?

I'd guess that he has a methodmap that wraps a client/player, so his this is not an int, but its value is a valid client index. But just guessing here.

[popey456963]

Originally Posted by KliPPy I'd guess that he has a methodmap that wraps a client/player, so his this is not an int, but its value is a valid client index. But just guessing here.

Yep! Otherwise, it complains that it's of type "Player", so I have to specify that it is an int. And I changed the stock to return a bool and everything worked, so thanks for the help guys.

[Peace-Maker]

I've fixed the return tag in the repo. You're very welcome to create a pull request next time you find a bug!

[popey456963]

Ah, nice @Peace-Maker! I must admit I didn't as I thought pull requests didn't seem to be being reviewed at the moment (e.g. 39, 45).

[Peace-Maker]

Those two add something game specific in two different ways and it wasn't decided how to handle game stocks yet.