[GuskiS]

I have made this API (look in attachment). What it basicly does is gives user weapon(lol), if it is normal, then it just checks that weapons ammo on Holster(removes from player) and on Deploy(gives back). You might ask why, it is because I wan't ammo to be weapon specific not player, so each weapon has it's own ammo count. Also, there is a way to give other features with API - models, attack speed, reload time etc.
So, where is the problem? It is with that many RegisterHam calls (I think), because when I register them in plugin_init, server crashes on map change (server starts normally, after map change = crash). So I though about adding it to plugin_precache, it fixed it a bit. Now it doesn't crash on every mapchange, but randomly and gives this crash log: https://forums.alliedmods.net/showpo...31&postcount=6

So my question is, what to change, so there won't be so many RegisterHam? Can I determine in CurWeapon event if it was holstered or deployed?

[Xalus]

Error: attachment not found.

[Nextra]

Ham doesn't crash because of too many RegiserHam calls and CurWeapon sucks. Fix your installation instead of searching for pointless workarounds.

[GuskiS]

Originally Posted by Xalus Error: attachment not found.

Hehe, forgot to add, thanks, added.

Originally Posted by Nextra Ham doesn't crash because of too many RegiserHam calls and CurWeapon sucks. Fix your installation instead of searching for pointless workarounds.

Well, you can try. For me (and others) it was crashing if I put all those Holster/Deploy in plugin_init.

[Nextra]

PHP Code:

#include <amxmodx>
#include <hamsandwich>

#pragma semicolon 1
#pragma ctrlchar '\'

new hpre, hpost, dpre, dpost;

public plugin_init() {
    register_plugin("Hamtest", "0.0.1", "Nextra");

    new wpnname[32];
    for(new wpn = CSW_P228; wpn <= CSW_P90; ++wpn) {
        if (get_weaponname(wpn, wpnname, charsmax(wpnname))) {
            for (new i = 0; i < 100000; ++i) {
                RegisterHam(Ham_Item_Holster, wpnname, "holster_pre_whynot");
                RegisterHam(Ham_Item_Deploy, wpnname, "deploy_pre_whynot");
                RegisterHam(Ham_Item_Holster, wpnname, "holster_post_whynot", .Post = 1);
                RegisterHam(Ham_Item_Deploy, wpnname, "deploy_post_whynot", .Post = 1);
            }
        }
    }

    register_clcmd("hamtest", "cmd_hamtest");
}

public holster_pre_whynot(const ent) {
    ++hpre;
}

public deploy_pre_whynot(const ent) {
    ++dpre;
}

public holster_post_whynot(const ent) {
    ++hpost;
}

public deploy_post_whynot(const ent) {
    ++dpost;
}

public cmd_hamtest(const client) {
    client_print(client, print_chat, "hpre: %d hpost: %d dpre: %d dpost: %d", hpre, hpost, dpre, dpost);
    hpre = hpost = dpre = dpost = 0;
} 


Doesn't crash.

[GuskiS]

Maybe you could test my plugin please? :/
Just put that cycle from plugin_precache to plugin_init.
Start server, join, change map = crash.

[Nextra]

It doesn't crash, and there's no reason it ever should. I have been running multiple plugins with similar or even more Ham usage for years, without any issues.

[GuskiS]

Did you really test it?

EDIT: Just tested myself, it crashes.

[Nextra]

Yes I tested it. And I put a 0 to 100 for loop around it just to be sure.

[GuskiS]

Well, that is just weird, cause I just installed new amxmodx, put default plugins and this one, and it still crashes.
Linux OS, newest HLDS, newest AMXX, Metamod v1.21p37.

Here is the crashing version.

Hmm, it seems that this crash happens when player decides to join.
Start server, player joins = works fines.
Server changes map, player joins = crash.
When I put it in plugin_precache - it works fine.