[xPaw]

Originally Posted by Sn!ff3r Cstrike module

whats wrong?

[DruGzOG]

Nothing so far lol

[crazyeffect]

Cool! Hot Plugin

[Sn!ff3r]

Originally Posted by xPaw whats wrong?

Anyone still use it? ;)

[hleV]

CVAR names are kinda strange to me.
If the amount of armor is set by sv_armor_amount, then armor type should be set by sv_armor_type. Though it has nothing to do with plugin optimization, that would only be less confusing.

Originally Posted by Sn!ff3r Anyone still use it? ;)

I can't see a reason for a person who is having a CS server to not have enabled CStrike module. It just sounds strange.

[ConnorMcLeod]

Originally Posted by Sn!ff3r Anyone still use it? ;)

Use fakemeta instead of cs_set_user_amor and you have to use 2 natives, 1 to set armor, another one to set armor type, so cstrike native should be 2 times faster or even more.
I don't even consider you need 3 more natives to send a ArmorType message.

-edit-
Seems that i was wrong, when you don't send a message, FM is faster, but you actually need to send the message.

PHP Code:

public prof( id )
{
    for(new i; i<10000; i++)
    {
        cs_set_user_armor(id, 100, CS_ARMOR_VESTHELM)
        fm_set_user_armor(id, 100.0, 2)
        fm_cs_set_user_armor(id, 100.0, 2)
    }
    return PLUGIN_HANDLED
}

fm_set_user_armor(id, Float:amount, type)
{
    set_pev(id, pev_armorvalue, amount)
    set_pdata_int(id, 112, type)
}

fm_cs_set_user_armor(id, Float:amount, type)
{
    set_pev(id, pev_armorvalue, amount)
    set_pdata_int(id, 112, type)

    if ( type )
    {
        message_begin( MSG_ONE_UNRELIABLE , gmsgArmorType , _ , id )
        write_byte( type == 2 ? 1 : 0 )
        message_end();
    }
} 


type |                             name |      calls | time / min / max
-------------------------------------------------------------------
   n |                  register_plugin |          1 | 0.000001 / 0.000001 / 0.000001
   n |                   register_clcmd |          1 | 0.000006 / 0.000006 / 0.000006
   n |                   get_user_msgid |          1 | 0.000001 / 0.000001 / 0.000001
   n |                cs_set_user_armor |      10000 | 0.006480 / 0.000001 / 0.000080
   n |                          set_pev |      20000 | 0.003963 / 0.000000 / 0.000001
   n |                    set_pdata_int |      20000 | 0.003909 / 0.000000 / 0.000068
   n |                    message_begin |      10000 | 0.002382 / 0.000000 / 0.000001
   n |                       write_byte |      10000 | 0.002153 / 0.000000 / 0.000000
   n |                      message_end |      10000 | 0.003001 / 0.000000 / 0.000023
   p |                      plugin_init |          1 | 0.000001 / 0.000001 / 0.000001
   p |                             prof |          1 | 0.005654 / 0.005654 / 0.005654
   f |                fm_set_user_armor |      10000 | 0.005802 / 0.000001 / 0.000046
   f |             fm_cs_set_user_armor |      10000 | 0.011227 / 0.000001 / 0.000045
0 natives, 0 public callbacks, 2 function calls were not executed.

Summary

type |                             name |      calls | time / min / max
-------------------------------------------------------------------
   n |                cs_set_user_armor |      10000 | 0.006480 / 0.000001 / 0.000080
   f |                fm_set_user_armor |      10000 | 0.005802 / 0.000001 / 0.000046
   f |             fm_cs_set_user_armor |      10000 | 0.011227 / 0.000001 / 0.000045
0 natives, 0 public callbacks, 2 function calls were not executed.

[anakin_cstrike]

Originally Posted by DarkGod Simple, useful, original. Good job.

PS: You don't have to check if the value is higher than 0 in that if statement, only for info this

[xPaw]

You're wrong, it could fuck up the players armor, armor type would be 0, and armor will se to 100? what it will be?

[ConnorMcLeod]

You should also check if amount set by plugin is > 0, because if you set armortype to 1 or 2, and set armorvalue to 0, then players won't be able to pick'up ground armors.

[anakin_cstrike]

Originally Posted by xPaw You're wrong, it could fuck up the players armor, armor type would be 0, and armor will se to 100? what it will be?

You're using clamp so "> 0" and "0" is the same thing in this case.