[Lee]

Originally Posted by connorr client_death is a csx forward so you need to include it.

No, you don't.

[Sam Tsuki]

Originally Posted by Lee No, you don't.

I thought so cuz it was working before I included the csx.inc :O

[Dores]

i'll test to make sure i'm right and to make you feel better (i'm reffering my other post with the usage)

EDIT: Works fine (the usage at least... haven't checked if plugin itself works..)

[ConnorMcLeod]

Originally Posted by Lee No, you don't.

Of course you have to !

[Dores]

Originally Posted by connorr Of course you have to !

The man says it works without it.... any reason he should include it?

[Lee]

Wow. I'm dumbfounded. How can both you and KWo not know how forwards work? Test it and then tell me otherwise.

[ConnorMcLeod]

Well, in order to this forward work, csx module need to run, so if csx module isn't declared in modules.ini, and if no other plugin make it autoload, this plugin won't work unless you include csx.
The only reason that this plugin work is that csx module is already running.

[Sam Tsuki]

Originally Posted by connorr Well, in order to this forward work, csx module need to run, so if csx module isn't declared in modules.ini, and if no other plugin make it autoload, this plugin won't work unless you include csx. The only reason that this plugin work is that csx module is already running.

But I think if you didn't include a module and used one of its forwards then it won't compile O.o

EDIT: No I guess it will comple...
Oh well
Will leave it at this

[ConnorMcLeod]

Originally Posted by Sam Tsuki But I think if you didn't include a module and used one of its forwards then it won't compile O.o EDIT: No I guess it will comple...

Yes, it compiles as your function is declared as 'public'

[Lee]

Connorr, 'should' is not the same as 'have to'. You were also wrong to say "Value 2 will never be detected" since he was checking two different cvars.

Connorr's not being cocky. He's rightly pointing out that someone with more experience would have done a much better job. There are still numerous errors.

• Scrap the loop below 'Check player names' in favour of cmd_target() or find_player().
• Player names can be 31 characters long.
• The fourth parameter to cmd_access() in give_nade() should be 3.
• You need to check that str_to_num() doesn't return 0.
• Don't return PLUGIN_HANDLED in forwards that can't be blocked.
• award_fb_amount is superfluous.
• Use 'break' to exit loops.
• There are many places where you should cache native return values.
• You should evaluate whether the first element of an array is false rather than using equal() to determine if a string is empty.
• You overwrite a cvar pointer (award_nades) with a value rather than updating the cvar itself.

Don't be discouraged. Everyone writes shoddy code at first. You should learn more before attempting to release plugins though.