RG_CBasePlayer_ImpulseCommands problem

[PurposeLess]

Hi, I wonder something about this command. When I use "client_impulse", it works perfect. But when I change to RG_CBasePlayer_ImpulseCommands, i have a problem.
When I use "impulse 201", it starts and never stops. How I can make it stop?

#include <amxmodx>
#include <engine>
#include <reapi>

public plugin_init() {
    RegisterHookChain(RG_CBasePlayer_ImpulseCommands, "CBasePlayer_ImpulseCommands", .post=false);
}

/* IT NEVER STOPS AND ALWAYS WRITE "Lololol" */
public CBasePlayer_ImpulseCommands(const id)
{
    static impulse;
    impulse = get_entvar(id, var_impulse);

    if(impulse == 201)
    {
        client_print(id, print_chat, "Lololol");
        return HC_SUPERCEDE;
    }
    return HC_CONTINUE;
}

/* Working perfecto ! :) */

public client_impulse(id, impulse)
{
    if(impulse == 201)
    {
        client_print(id, print_chat, "Lololol");
        return 1;
    }
    return 0;
}

 

[twisterniq]

It works as expected. Why should client_print(id, print_chat, "Lololol"); not work? You print a message before you supercede.

 

[PurposeLess]

w0w,
It works anyway, it does not stop writing "Lololol". When you use "impulse 201", it always continues. It force you to use impulse 201.
Lemme explain;

+ w client_impulse;
Player used impulse 201
Lololol
Not sprayed [return 1]

+ w RG_CBasePlayer_ImpulseCommand;
Player used impulse 201
Lololol
Not sprayed [return 1]
Lololol
Not sprayed [return 1]
Lololol
Not sprayed [return 1]
Lololol
Not sprayed [return 1]
... (continues)

 

[twisterniq]

Yes, it's the way this/each hook works.

 

[PurposeLess]

Can't we make it working as client_impulse?
For example; I am using client_impulse in "CS GO Spray plugin" and not able to change it to reapi. Weird.

 

[Mistrick]

You should set var_impulse to 0 before supercede.

 

[PurposeLess]

@Mistrick,
So, for optimizing which one should I use?
Engine client_impulse or Reapi CBasePlayer....blabla
Is there any difference?

#include <engine>
#include <reapi>

public plugin_init() {
    RegisterHookChain(RG_CBasePlayer_ImpulseCommands, "CBasePlayer_ImpulseCommands", .post=false);
}

public CBasePlayer_ImpulseCommands(const id)
{
    static impulse;
    impulse = get_entvar(id, var_impulse);

    if(impulse == 201)
    {
        client_print(id, print_chat, "Lololol");
        set_entvar(id, var_impulse, 0);
        return HC_SUPERCEDE;
    }
    return HC_CONTINUE;
}

public client_impulse(id, impulse)
{
    if(impulse == 201)
    {
        client_print(id, print_chat, "Lololol");
        return 1;
    }
    return 0;
}

 

[Mistrick]

I think both are the same.