[Emp`]

	register_message(get_user_msgid("Money"),"hook_money")
	
	onoff = register_cvar("amx_mhe_money","1")
	speed_per_sec = register_cvar("amx_mhe_money_speed","10")
	
	RegisterHam(Ham_Spawn,"player","player_prespawn",0)
	RegisterHam(Ham_Spawn,"player","player_spawn",1)
	
	money = get_user_msgid("Money")

->

	money = get_user_msgid("Money")
	register_message(money,"hook_money")
	
	onoff = register_cvar("amx_mhe_money","1")
	speed_per_sec = register_cvar("amx_mhe_money_speed","10")
	
	RegisterHam(Ham_Spawn,"player","player_prespawn",0)
	RegisterHam(Ham_Spawn,"player","player_spawn",1)

[shine771]

Didn't notice that lol. Thanks.

[jim_yang]

In your plugin there is only one set task in use, you don't need to do the taskid calculation, just use id
second, you can use this instead of task_exists in your case

new g_task[33]
public test2(id)
{
    g_task[id] = set_task(1.0, "test3", id, _, _, "b")
    return 1
}
public test3(id)
{
    static i = 0
    if(i++ < 5)
    {
         client_print(id, print_chat, "%d", i)
         return
    }
    if(g_task[id] && remove_task(id))
         g_task[id] = 0
}

besides, nice job

[shine771]

task_exists is more simple. No extra vars.. Yea i know, i should use remove_task(id) only. When i started making this plugin, i dunno why, but i was thinking of using 2 tasks with id. So i was lazy to edit it after i finished it.. anyway. If i'll make another version, i'll fix that.

Thanks all...

[F4RR3LL]

good.

[jim_yang]

http://www.sourcemod.net/devlog/?p=118

[Hawk552]

There are a few mistakes and optimizations that can be made, however it is acceptable.

If you would like any information regarding possible adjustments you could make or things you could do to make this better, please feel free to post here or PM me.

Approved.

[P4rD0nM3]

Kinda like a trickling effect am I right?

[lIIusion]

Nice, this is on my server now!

[Hawk552]

Changes have been requested via PM.

See your other plugin for stylistic changes.

remove_task(id+TASK_EFFECT)

Unnecessary instructions. You never use the taskid value of id, so there's no reason to add a unique identifier for it.

set_task(0.01,"money_effect",id+TASK_EFFECT,_,_,"b")

Read my article on the set_task() function. The timer value is clamped at 0.1, so if you want something lower you'll have to use another method.

message_begin(MSG_ONE,money,_,id)

Use MSG_ONE_UNRELIABLE.

stock set_plr_money(id,Money,flash=0) {

Read my article on stocks.

This plugin is pretty good. It's a bit better than your other one, but still has some stylistic issues of inconsistency and poor readability.