[abdul-rehman]

public client_connect(id) 
    has_deagle[id] = false 

public client_disconnect(id) 
{ 
    has_deagle[id] = false 
} 

public death(id) 
{
    has_deagle[id] = false 
}
public zp_user_infected_post(id, infector)
{
    has_deagle[id] = false 
}
public zp_round_started ( GameMode, id )
{
    has_deagle[id] = false 
}

to

public client_disconnect(id) 
    has_deagle[id] = false 

public death(id) 
    has_deagle[id] = false 

public zp_user_infected_post(id, infector)
    has_deagle[id] = false

And if you want to remove it on round start you need to loop through all players using register_event and dont use zp_round_started to set it false since sometimes the player id is 0 in the second parameter


And change this:

PHP Code:

entity_set_vector(this , EV_VEC_punchangle , fVec); 


To this:

PHP Code:

pev( this , pev_punchangle , fVec ) 


So you can remove the engine module

And remove this:

PHP Code:

 set_user_health 


With this:

PHP Code:

 set_pev( id, pev_health, ... ) 


To retrieve player's health use pev( id , pev_health, Float:Value )

So you can remove the fun module as well
So the includes will look like this:

#include <amxmodx> 
#include <zombieplague> 
#include <hamsandwich> 
#include <cstrike> 
#include <fakemeta>
#include <xs> 

Instead of this:

#include <amxmodx> 
#include <zombieplague> 
#include <hamsandwich> 
#include <cstrike> 
#include <fakemeta>
#include <xs>
#include <fun>
#include <engine> 

[papyrus_kn]

L 09/14/2010 - 19:40:07: [AMXX]    [0] Dynamic_deagle.sma::HamPreDamage (line 74)
L 09/14/2010 - 19:40:34: [AMXX] Displaying debug trace (plugin "Dynamic_deagle.amxx")
L 09/14/2010 - 19:40:34: [AMXX] Run time error 4: index out of bounds 
L 09/14/2010 - 19:40:34: [AMXX]    [0] Dynamic_deagle.sma::HamPreDamage (line 74)

[abdul-rehman]

Just add a check in ham take damage that whether player is a connected and is also valid ( 1 <= id <= 32 )

[teobrvt1995]

nice

[Welgericht]

Update to ver 1.0.1
check first post for more information

[Excalibur.007]

Simple yet good.
@abdul
Btw, I prefer { } no matter the public function has only 1 line because the readability is better although 2 lines is added. Everybody has different style of coding so I'm not saying anything more. Just to say { } is better than lesser than 2 lines.

[abdul-rehman]

Originally Posted by Excalibur.007 Simple yet good. @abdul Btw, I prefer { } no matter the public function has only 1 line because the readability is better although 2 lines is added. Everybody has different style of coding so I'm not saying anything more. Just to say { } is better than lesser than 2 lines.

I agree that its better but i also like it like this:

public client_is_dancing( no_one ) g_isalive[ no_one ] = true

[Shaw]

Small copy of another plug-in. Congrats on originality. (sarcasm)

[Welgericht]

Shaw Yeah i agree, it is not full of originality or smth. else but Not everybody wants to have plug-in SAP Bank on their server.
I don't say it is bad, but it creates a big disbalans, and is not something special.

[mottzi]

Your
Indentation

is horrible