[Exolent[jNr]]

Umm... how would a fakemeta version of a cstrike module work in another mod?

[danielkza]

One thing you can do using fakemeta'ed natives is include support for mod specific functions without tying the plugin to a game. For instance, you can check what game is running and choose beetween cs_set_user_money or ts_set_user_money, or just disable the feature. If you include one of the modules, the plugin simply wouldn't load at other games.

Here is an example: If I used the cstrike module instead of fakemeta, this command wouldn't work on other mods:

stock fm_cs_set_user_armor(id, iValue, iType = CS_ARMOR_VESTHELM)
{
    set_pev(id, pev_armorvalue, float(iValue))
    
    // Counter-Strike
    static bCStrike
    if(!bCStrike)
        bCStrike = GetGameRunning(GAME_CSTRIKE) ? 1 : -1
    
    if(bCStrike == 1)
    {
        set_pdata_int(id, OFFSET_ARMORTYPE, iType)
        
        static msgArmorType
        if(!msgArmorType)
            msgArmorType = get_user_msgid("ArmorType")
        
        if(msgArmorType)
        {
            message_begin(MSG_ONE, msgArmorType, _, id)
            write_byte(iType == CS_ARMOR_VESTHELM ? 1 : 0)
            message_end()
        }
    }
    
    return true
}

public Armor_Command(id,level,cid)
{
    if(!cmd_access(id,level,cid,3) || !_CmdTarget(1, id, (CMDTARGET_OBEY_IMMUNITY|CMDTARGET_ALLOW_SELF|CMDTARGET_ONLY_ALIVE)))
        return PLUGIN_HANDLED

    new iArmorType = CS_ARMOR_KEVLAR
    if(UTIL_ReadArgBool(3, id) == 1)
        iArmorType = CS_ARMOR_VESTHELM
    
    new iArmor = 0
    if(!UTIL_ReadArgInt(2, iArmor, CHECK_NONZERO))
        console_print(id, "%L", "ARG_NUM_INVALID", 2)
    else
    {
        new iPlayer, iTempArmor
        
        for(new i=0; i < g_iPlayersNum; i++)
        {
            iPlayer = g_iPlayers[i]
            
            iTempArmor = max(get_user_armor(iPlayer) + iArmor, 0)
            fm_cs_set_user_armor(iPlayer, iTempArmor, iArmorType)
        }
        
        UTIL_PrintActivity(id, (iArmor > 0) ? "ARMOR_GIVE" : "ARMOR_TAKE", g_szTarget, abs(iArmor))
    }
    
    return PLUGIN_HANDLED
}

[Emp`]

Originally Posted by danielkza One thing you can do using fakemeta'ed natives is include support for mod specific functions without tying the plugin to a game. For instance, you can check what game is running and choose beetween cs_set_user_money or ts_set_user_money, or just disable the feature. If you include one of the modules, the plugin simply wouldn't load at other games.

That's what these:

	set_native_filter( "native_trapper" );
	set_module_filter( "module_filter" );

are for

[joaquimandrade]

Originally Posted by stupok So The Specialists has the same offset for money as Counter-Strike? It (probably) doesn't, and that's what vittu is getting at. Each mod has its own offsets and little intricacies. Using fakemeta isn't making anything more compatible. I suppose you could argue that with fakemeta, you would only have to change a few #defines that declare the offsets, where if you use cs_get_user_money, you would have to do a "replace all" with ts_getusercash. Further, maybe some mods don't have a function, so you would need to use fakemeta. That's a negligible benefit, in my opinion. I think a trend is exactly what it is.

You are right. I didn't played nor programmed for another mod so, i don't know what i'm talking about.The Cstrike module had gained a negative connotation but, using it, for the majority of the cases, its a better solution than fakemeta. (It does the same as a fakemeta equivalent, but in C++, so there isn't the expense of calling natives from Pawn)

[Hawk552]

Originally Posted by vittu Some people just got on some crazy fakemeta high and thought everything needed to be converted to it to use less modules because they were told doing so was optimizing. Not specifically talking about this thread or anyone who has posted in this thread.

Completely agreed. I went on a rant like this a while ago. Using FakeMeta in place of cstrike is a terrible idea. If the offsets ever change, the AMXX team will jump right on it and release a new cstrike module right away. This would require anyone using this header to update it themselves or hope that you're still around to do it. There are many, many other problems but most have already been covered in this thread.

EDIT: Here it is:
http://forums.alliedmods.net/showthread.php?p=734716

It's not directly related to the cstrike module, but many of the concepts still apply.

[joaquimandrade]

Profile results

[Arkshine]

Thanks for the bench joaquimandrade! I'm not surprised by the result at all.

[Hawk552]

Not surprising at all. Even if it was faster, the other consequences of using this would outweigh the benefits.

[Empowers]

Error: Undefined symbol "m_iWeaponFiremode"

[Exolent[jNr]]

Sorry about that. It is fixed now.