[soccdoodcss]

That's a god idea... Ill look into it. Also, I wonder if there are any gladiator-like shield models.

[soccdoodcss]

Sorry to bump yet again. But I was wondering is anyone was willing to share there opinions on this plugin. Any suggestions are also more than welcome.

[Da_sk8rboy]

why not remove all the other plugins besides the newest & updated version?

[Hawk552]

Originally Posted by Da_sk8rboy why not remove all the other plugins besides the newest & updated version?

Do this.

get_pcvar_num(gm_level) !=0

Although this technically is the same thing and I'm pretty sure it gets optimized out, it looks nicer if you do this instead:

get_pcvar_num(gm_level)

Similarly, this:

get_pcvar_num(gm_on) == 0

can be shortened to:

!get_pcvar_num(gm_on)

new killer = read_data(1)
    new victim = read_data(2)
    new killername[31]
    get_user_name(killer,killername,31)
    new victimname[31]
    get_user_name(victim,victimname,31)

There are 2 things you can do with this. One is mandatory, the other isn't.

You never checked if the killer is actually connected before you got his name. If anyone does kill in console, dies for an unknown reason, or anything like that, they will trigger a runtime error.

It also looks nicer, although it's the same functionality, to do this instead:

new killer = read_data(1),victim = read_data(2),killername[31]
    get_user_name(killer,killername,31)
    new victimname[31]
    get_user_name(victim,victimname,31)

Since it looks like you're using the style where you drop brackets whenever possible, you should be consistent. Places like this:

public end_vote()
{
    if (voted==0)
    {
        if (vote_yes > vote_no)
        {
            set_vote_yes()
        }
        else if (vote_no >= vote_yes)
        {
            set_vote_no()
        }
    }
}

Might look like this instead:

public end_vote()
    if (voted==0)
        if (vote_yes > vote_no)
            set_vote_yes()
        else if (vote_no >= vote_yes)
            set_vote_no()

Places like this:

public print_vote()
{
    min_votes = num/2
    client_print(0,print_chat,"[GM] Yes: %d      No: %d      Needed: %d",vote_yes,vote_no,min_votes)
    set_vote()
}

Should be private instead (i.e. remove the "public"). The rule of thumb is that any function which is never used in quotes and is not defined as a forward should be private.

[soccdoodcss]

Alright, thanks a lot. Optimizing now.

Edit: I don't fully get the whole private/public thing yet. I will look into understanding it. For now, i fixed it up and removed the public from the 2 print vote functions. Fixed other things.

Edit 2: Fixing: Sometimes on round start you wouldnt start with a shield. Had some bugs, fixed them. If anyone finds anymore, tell me.

Edit 3: Ive decided not to set a model for the knife. Mostly because i havent found any appropriate models.

[1NiteStand]

u should make it so that u can use knife and guns so when u are holding a knife this mod is active and when ur holding a gun this plugins is disabled. u know wat i mean?

[vvg125]

That ultimately defeats the point of the plugin? This plugin is made so it's like a knife arena sort of thing. What's the point if people can use guns?

[soccdoodcss]

Exactly, it is a knife only mod. I think pistols with shield would be fun, but only if it a separate plugin, or a different setting on this plugin.

[kingzope]

Wow I will have to try this out. Great idea and +karma man.

[soccdoodcss]

Originally Posted by kingzope Wow I will have to try this out. Great idea and +karma man.

Thanks, hope you are still fond of it after trying.
BTW: If anyone has any ideas or suggestions, I would gladly consider them. You know where to call.