[Phantom Warrior]

We posted at same time

[stigma]

Okay.. you mean then my give_item should be in the "if"?

And how do i get that last death message..?

Edit:

Avalanche, i've tryed to put into a variable, but that ave me the same error too.

[Phantom Warrior]

Now, copy and paste the full code(With changes)

[stigma]

#include <amxmodx>
#include <fun>
#define PLUGIN "New Plugin"
#define VERSION "1.0"
#define AUTHOR "Author"

new time = get_gametime();
public plugin_init() {

register_plugin(PLUGIN, VERSION, AUTHOR)
register_event("DeathMsg","giveItem","a")

}

public giveItem() {
new killer = read_data(1)
new victim = read_data(2)
new kName[32], vName[32]

get_user_name(killer,kName,31)
get_user_name(victim,vName,31)

client_print(0,print_chat,"Killer: %s - Victim: %s",kName,vName)
if (killer != victim) {
if (time < 0,2) {
give_item(killer,"weapon_hegrenade")
}
}
}

[stigma]

Sorry for double posting, but if i edit in a post where there is some code it just fucks up..

But what i dont understand is the part with the lastDeathMsg[victim]

[XxAvalanchexX]

This is how I would implement it. Sorry if I messed up your formatting.

#include <amxmodx>
#include <fun>

#define PLUGIN "New Plugin"
#define VERSION "1.0"
#define AUTHOR "Author"

new Float:lastDeathMsg[33] // bugfix

public plugin_init() {
    register_plugin(PLUGIN, VERSION, AUTHOR)
    register_event("DeathMsg","giveItem","a")
}

public giveItem() {
    new killer = read_data(1)
    new victim = read_data(2)

    // bugfix
    new Float:now = get_gametime()
    if(lastDeathMsg[victim] - now < 0.2)
        return;
    lastDeathMsg[victim] = now

    new kName[32], vName[32]

    get_user_name(killer,kName,31)
    get_user_name(victim,vName,31)

    client_print(0,print_chat,"Killer: %s - Victim: %s",kName,vName)
    if (killer != victim) {
        give_item(killer,"weapon_hegrenade")
    }
}

[stigma]

Okay thanks alot.. But isent it necessary to use the {} when you make the if's?

And what was it the lastDeathMsg, was doing?

[XxAvalanchexX]

You only need to use curly braces {} if the "if" statement contains multiple lines of code to be executed. Since "return" is just one line of code, you don't need the braces. For example:

if(statement) line1 // line1 is executed if statement is true

if(statement)
   line1 // line1 is executed if statement is true

if(statement)
   line1 // line1 is executed if statement is true
line2 // line2 is always executed, because there are no curly braces to group this with the if above, and it's not the first line after the if statement

if(statement)
{
     line1 // line1 is executed if statement is true
     line2 // line2 is executed if statement is true
}

As for lastDeathMsg, what it is doing is recording at what time a player dies. Then, when a death message comes up, it compares that time with the time the death message was called. If this death message was called within 0.2 seconds of the last one, then it must be a glitchy death message from give_item, because no one naturally dies twice within 0.2 seconds.

[stigma]

Okay.. Thanks alot!