[danielkza]

Originally Posted by Exolent[jNr] I'm going to add on to what arkshine said: new bool:is_glowing[33][2049] You should just do a bit sum. Code: new g_glowing_ents[33]; // to set a ent glowing for the player g_glowing_ents[id] |= (1<<ent); // to remove g_glowing_ents[id] &= ~(1<<ent); // to check if( g_glowing_ents[id] & (1<<ent) ) I think that should work fine. Unless I'm being stupid and there is a limit for bits. If I am being stupid, use a dynamic Array, and push/delete entities when they are(n't) glowing. -------------------------------- For your cvar with color: You call that so many times in the plugin. You call it every AddToFullPack frame, and every time the HUD is shown a message (every 0.1 seconds when enabled). Instead, hook the new round event and store the colors in global variables each round. EDIT: Also, I don't see anywhere that you set the glowing entity to remove the glow if player moves away from the weapon.

Max 32 bits (0-31) for x86 and 64 (0-64) for x64. And In my opinion, using more memory in arrays in favor of less CPU use is better, since memory is much cheaper than new processors.

[Exolent[jNr]]

I edited again before I even saw your post.

[shine771]

That is_glowing array i had fixed some time ago. Just added that on round_start it writes cvars in global variables. Fixed the version xd too. Made classname[33] a static variable.
I never thought that my code would be That much ****** up... :O.

EDIT: @arkshine - how often you change weapons in cs, 1 round? Less than 3.

[letgoofmyeggs]

lol so can yu set the cvar so yu can picc up a gun all tha way across the map?

[Exolent[jNr]]

Everything seems okay except for one thing you overlooked:

public addtofullpack(es_handle,e,ent,id,hostflags,player,pSet) {
    if(!is_user_alive(id) || !get_pcvar_num(onoff) || !get_pcvar_num(glow) || (id==ent)) return FMRES_IGNORED
    
    if(is_glowing[id]==ent) {
        new temp_rgb[12],temp_rgb2[3][4],rgb[3]
        
        get_pcvar_string(glow_color,temp_rgb,11)
        
        parse(temp_rgb,temp_rgb2[0],3,temp_rgb2[1],3,temp_rgb2[2],3)
        
        rgb[0]=red
        rgb[1]=green
        rgb[2]=blue
        
        set_es(es_handle,ES_RenderMode,kRenderNormal)
        set_es(es_handle,ES_RenderFx,kRenderFxGlowShell)
        set_es(es_handle,ES_RenderAmt,16)
        set_es(es_handle,ES_RenderColor,rgb)
    }
    return FMRES_IGNORED
}

You still retrieved the cvar in the AddToFullPack, but you did change it to the red, green, blue instead of the cvar's values that were parsed.

Just delete the parsing and you're good.

public addtofullpack(es_handle,e,ent,id,hostflags,player,pSet) {
    if(!is_user_alive(id) || !get_pcvar_num(onoff) || !get_pcvar_num(glow) || (id==ent)) return FMRES_IGNORED
    
    if(is_glowing[id]==ent) {
        new rgb[3]
        
        rgb[0]=red
        rgb[1]=green
        rgb[2]=blue
        
        set_es(es_handle,ES_RenderMode,kRenderNormal)
        set_es(es_handle,ES_RenderFx,kRenderFxGlowShell)
        set_es(es_handle,ES_RenderAmt,16)
        set_es(es_handle,ES_RenderColor,rgb)
    }
    return FMRES_IGNORED
}

[shine771]

Oh.. omg. Forgot to remove that parsing part :O. I was in a hurry. Seems like everything is ok now.

[Arkshine]

EDIT: @arkshine - how often you change weapons in cs, 1 round? Less than 3.

Depending the map type, a lot of times. Even on "normal" map, some people like to change often weapons. :p

Anyway don't worry about what I say, I will do my way for my server, that's all. ^^

[rpm]

nice!

[shine771]

Someone could test it?

[crazyeffect]

This works!!