[You9]

My zombie server after refresh restart use 300mb RAM, but 2-3 hours later it uses 700mb RAM. So i used sm_dump_handles, but there is only "410494 bytes" used, that should not be too much?

But still, i have much-much IBaseMenu lines there, are there anyway i can close those?
Every menu has MenuAction: delete menu; btw

Spoiler

PHP Code:

L 01/04/2017 - 20:29:53: 0x1e360273CZombie.smx IBaseMenu           225       
L 01/04/2017 - 20:29:53: 0xaf6e0274CZombie.smx IBaseMenu           225       
L 01/04/2017 - 20:29:53: 0x9b510275CZombie.smx IBaseMenu           225       
L 01/04/2017 - 20:29:53: 0x059b0276CZombie.smx IBaseMenu           225       
L 01/04/2017 - 20:29:53: 0x05570277CZombie.smx IBaseMenu           225       
L 01/04/2017 - 20:29:53: 0x05a70278CZombie.smx IBaseMenu           225       
L 01/04/2017 - 20:29:53: 0x053f0279CZombie.smx IBaseMenu           225       
L 01/04/2017 - 20:29:53: 0xad67027aCZombie.smx IBaseMenu           225       
L 01/04/2017 - 20:29:53: 0x0527027bCZombie.smx IBaseMenu           225       
L 01/04/2017 - 20:29:53: 0x1f7a027cCZombie.smx IBaseMenu           225       
L 01/04/2017 - 20:29:53: 0x0533027dCZombie.smx IBaseMenu           225       
L 01/04/2017 - 20:29:53: 0xe822027eCZombieNade.smx IBaseMenu           459       
L 01/04/2017 - 20:29:53: 0x2fb3027fCZombie.smx IBaseMenu           225       
L 01/04/2017 - 20:29:53: 0xcc430280CZombie.smx IBaseMenu           225       
L 01/04/2017 - 20:29:53: 0x9b3e0281CZombie.smx IBaseMenu           225       
L 01/04/2017 - 20:29:53: 0xb8ee0282CZombie.smx IBaseMenu           225       
L 01/04/2017 - 20:29:53: 0x51890283CZombie.smx IBaseMenu           225       
L 01/04/2017 - 20:29:53: 0x19460284CZombie.smx IBaseMenu           225       
L 01/04/2017 - 20:29:53: 0x96be0285CZombie.smx IBaseMenu           225       
L 01/04/2017 - 20:29:53: 0x2c660286CZombie.smx IBaseMenu           225       
L 01/04/2017 - 20:29:53: 0x90080287CZombie.smx IBaseMenu           225       
L 01/04/2017 - 20:29:53: 0xa5580289CZombie.smx IBaseMenu           225       
L 01/04/2017 - 20:29:53: 0xfb93028aCZombie.smx IBaseMenu           225       
L 01/04/2017 - 20:29:53: 0x5141028bCZombie.smx IBaseMenu           225       
L 01/04/2017 - 20:29:53: 0xa900028cCZombie.smx IBaseMenu           225       
L 01/04/2017 - 20:29:53: 0xb57b028dCZombie.smx IBaseMenu           225       
L 01/04/2017 - 20:29:53: 0xd594028eCZombie.smx IBaseMenu           225       
L 01/04/2017 - 20:29:53: 0xb521028fCZombie.smx IBaseMenu           225       
L 01/04/2017 - 20:29:53: 0xd47b0290CZombie.smx IBaseMenu           225       
L 01/04/2017 - 20:29:53: 0x36ca0291CZombie.smx IBaseMenu           225       
L 01/04/2017 - 20:29:53: 0xd0c70292CZombie.smx IBaseMenu           225       
L 01/04/2017 - 20:29:53: 0x51310293CZombie.smx IBaseMenu           225       
L 01/04/2017 - 20:29:53: 0x46a00294CZombie.smx IBaseMenu           225 


[Mitchell]

Tell the creator of the plugin to close the damn handles when the client closes the menu.
didn't see that the menu handles are being closed on End.

[You9]

Originally Posted by Mitchell Tell the creator of the plugin to close the damn handles when the client closes the menu.

delete menu; = CloseHandle(menu);?

[Chaosxk]

Originally Posted by You9 delete menu; = CloseHandle(menu);?

Yes, show us the code...you might of used it improperly.

[Mitchell]

deleteing the menu option on every MenuAction is a bad idea, and could crash the server (iirc) or cause plugin errors.

[You9]

Originally Posted by Chaosxk Yes, show us the code...you might of used it improperly.

It's custom plugin and has lot's of lines
But here's 1 menu handler example:

Spoiler

PHP Code:

public int MenuHandler_PrimaryRifle(Menu menu, MenuAction action, int client, int item) 
{
    switch(action){
        case MenuAction_Select: {
            int chosen = GetMenuInt(menu, item);
            Cookie_Rifle[client] = chosen;
            SetCookieInt(client, CookHandler_Rifle, chosen);
            CPrintToChat(client, CHAT_TAG, "Changed option rifle1");
        }
        
        case MenuAction_End: delete menu;
    }
} 


Originally Posted by Mitchell deleteing the menu option on every MenuAction is a bad idea, and could crash the server (iirc) or cause plugin errors.

Even alliedmods wiki is using "delete menu;", and i thought it's the best way

[Mitchell]

What about the creation of the menu handle. are you setting it so cancel ends the menu? I'm not sure what the setup of the menus even are and how many menus you are creating. You could be creating a very large menu for each of the clients, instead of creating one menu and handling it through the draw actions.

[You9]

That is 1 example of all menus, and i have lot's of menus, but they all have same structure.

PHP Code:

Menu rmenu = new Menu(MenuHandler_PrimaryRifle);
rmenu.SetTitle("Rifle:");
rmenu.AddItem("1", "M4A1-s");
rmenu.AddItem("2", "AK-47");
rmenu.AddItem("3", "Famas");
rmenu.AddItem("4", "Galil");
rmenu.AddItem("5", "AWP");
rmenu.AddItem("6", "MP7");
rmenu.ExitButton = true;
rmenu.Display(client, 35);

public int MenuHandler_PrimaryRifle(Menu menu, MenuAction action, int client, int item) 
{
    switch(action){
        case MenuAction_Select: {
            int chosen = GetMenuInt(menu, item);
            Cookie_Rifle[client] = chosen;
            SetCookieInt(client, CookHandler_Rifle, chosen);
            CPrintToChat(client, CHAT_TAG, "Changed option rifle1");
        }
        
        case MenuAction_End: delete menu;
    }
} 


[Mitchell]

You are creating a static menu for each client which displays the same information. Try creating the menu on pluginstart and then when you want to display it use the handle you created in pluginstart.

[You9]

What should i do with the "not static" menus? I mean like where every time you open menu there is all terrorist team players, or database data what is always new.