[abdul-rehman]

It would be better if you would just cache whether the user is connected or not:

// Using bits will be far more effecient
new g_bitConnectedPlayers
#define MarkUserConnected(%0)   ( g_bitConnectedPlayers |= ( 1 << ( %0 & 31 ) ) )
#define ClearUserConnected(%0)  ( g_bitConnectedPlayers &= ~( 1 << ( %0 & 31 ) ) )
#define IsUserConnected(%0)     ( g_bitConnectedPlayers & ( 1 << ( %0 & 31 ) ) )

// Put this in thos forwards
public client_connect(id) MarkUserConnected(id)
public client_disconnect(id) ClearUserConnected(id)

some_function( id )
{
    // To check if user is connected
    if ( IsUserConnected(id) )
    {
        // ...
    }
}

[Nightfall1]

abdul-rehman I lose my brain some where, I don't understand anything :O, that is hard level slow down a bit

[NiHiLaNTh]

Bit-shifts are not so hard.Maybe this tutorial will help you to understand bit shift stuff.

Edit.

public reset_cvar()
{
    for(new i = 1; i <= g_maxplayer; i++)
    {
    g_IronJump[i] = false
    g_IronMan[i] = false
    }
}

could be also done like this

public reset_cvar()
{
    arrayset( g_IronJump, false, 33 )
    arrayset( g_IronMan, false, 33 )
}

[abdul-rehman]

Originally Posted by Nightfall1 abdul-rehman I lose my brain some where, I don't understand anything :O, that is hard level slow down a bit

If you dont understand bits, then use booleans instead to cache when da player was connected:

new bool:g_bIsConnected[33]

public client_connect( id ) g_bIsConnected[ id ] = true

public client_disconnect( id ) g_bIsConnected[ id ] = false

// To check is player is connected
some_function( id )
{
    if ( g_bIsConnected[id] )
    {
         // do something
    }
}

[Arkshine]

Caching the value for that is pointless. That's not like the forward will be called very often and all the time, so it's fine as it is. He should concentrate first to clean up the code, fixing things as suggested, etc, before trying to optimize the plugin.

[ANTICHRISTUS]

there is a iron man model in this pack http://www.mediafire.com/?1brhtdl6a5zjodt#1
it will be nice with this plugin

[ConnorMcLeod]

.Don't use engfunc(EngFunc_PrecacheXXX
.Use Ham_Deploy to set knife model or at list put filters in the register_event line

PHP Code:

    RegisterHam(Ham_Touch, "player", "fw_IronTouch")
    RegisterHam(Ham_Touch, "worldspawn", "fw_IronTouch") 


.I'm curious how you can use the same handler for those 1 forwards, fix it.

PHP Code:

    register_logevent("ResetStuffs", 2, "1=Round_Start")
    RegisterHam(Ham_Spawn, "player", "ResetStuffs") 


.Same here, you can't use the same callback, fix it.



.use_iron function doesn't need to be public, it shouldn't even be an extra function as it is only called from one place, fplease fix it.

PHP Code:

set_uc( UC_Handle, UC_Buttons, (buttons & IN_ATTACK & IN_ATTACK2)); 


.You don't understand what you are doing here.

PHP Code:

        for(new victim = 1; victim <= g_maxplayer; victim++)
        {
            IronDamage(victim, iron, pointer1, pointer2)
            IronKnockback(victim, iron, origin, Float:pointer3, pointer1, pointer2) // LOL THIS IS REALY WORK ? or.... NOT :))
            IronKnockback2(victim, iron, pointer3, pointer1, pointer2) // assumed this one it work
        } 


.Use get_players native, then you don't have to check in all the used functions is victim is alive.

[ConnorMcLeod]

No changes nor confirmation you work on it, unapproved.

[newzombieserveruk]

Has anybody got a fix for this