[Drixevel]

well it's too bad i don't know how to make plugins.

[Afronanny]

You should learn. It's not really that hard. Pawn is a pretty intuitive first language for someone new to coding server plugins.

[noodleboy347]

Originally Posted by asherkin From the code, it looks like it's trying to get the health of world when you get hurt from fall damage. :p Easy fix.

Will do.

Originally Posted by Afronanny You should learn. It's not really that hard. Pawn is a pretty intuitive first language for someone new to coding server plugins.

It is really easy actually, it's the first language I learned on my own.

[retsam]

You got some coding mistakes in this for sure...just fyi. Briefly looking over your other new plugin, you got quite a few mistakes on that one too.

[noodleboy347]

Originally Posted by retsam You got some coding mistakes in this for sure...just fyi. Briefly looking over your other new plugin, you got quite a few mistakes on that one too.

Great, care to share? Or do you just want to keep that to yourself?

[noodleboy347]

Version 1.0.1 - [2/21/10] -Fixed invalid client errors

[retsam]

Well ok. I thought they were kinda obvious and figured youd fix them.

Briefly looking at this one:

Hurt hook:
- You need to check if the clients are valid. You have zero checks in there, which would mean many errors, which is probably what youre seeing. Checks for (<= 0, > maxclients, ingame, and alive checks, etc)

Cosmetic Suggestions:
- Dont use client for attacker and hurt for client. Common practice is client is victim and attacker is attacker. Makes it confusing to read, although thats more cosmetic than an issue..
- An enabled check should be above everything. It should be at the top of the hurt hook. Silly to getconvarint on it 3 times under the hurt hook after its already running code.

Cvars:
- Need to set cvars to INVALID_HANDLE

Suggestion:
- Should learn to use globals for cvars and bools. You shouldnt need to getconvarint that many times. I see you do that with almost every plugin of yours. Infact, theres no reason to even getconvarint like that. Just get it once, and put everything else under it....

[Chris-_-]

Don't pick to much, retsam

Originally Posted by retsam Suggestion: - Should learn to use globals for cvars and bools. You shouldnt need to getconvarint that many times. I see you do that with almost every plugin of yours. Infact, theres no reason to even getconvarint like that. Just get it once, and put everything else under it....

This and HookConVarChange, best solution

[noodleboy347]

You guys are acting like I'm pro at Sourcepawn. I'm not. :/

[retsam]

Shiet, im not either heh. Im only a novice as this stuff. You asked. Just saying, if you wonder where the errors come from, its the above stuff I just commented on.

Some of the code issues(not necessarily this plugin) can prevent your plugins from getting approved.