[Jhob94]

Originally Posted by Arkshine Some thoughts while I'm passing here: Use formatex. Actually since you don't use arguments, I don't see why you're using vformat. No reason to use static on admins. This would be welcomed you add some comments in your plugin about what does the plugin, listing/explaining cvars.

Made even better

[Arkshine]

Well, if I had one last complaint, is PLUGIN and VERSION are used three times, you should use new const here. Just better memory management.

[Jhob94]

Originally Posted by Arkshine Well, if I had one last complaint, is PLUGIN and VERSION are used three times, you should use new const here. Just better memory management.

I never really understood the difference between using define and const for this things. I know this isn't a scripting help forum, but can you explain me or pm why?
Once i used scripting help forum and i got solution but then someone asked why i did like that and not the other way and i didn't knew why. Since that day i like to know the whys. Some people got "spammed" with stupid scripting questions

Edit:
Updated! And i have a feeling that i am gonna get banned

[Backstabnoob]

It's simple, define is a preprocessor directive which is replaced during compilation. So if you #define SOME_STRING "some_string" and use it either in a function that is called frequently or on multiple places, the memory will be allocated (and free'd) each time the string is used, which is a waste of computing power (not that it really matters, though).

Where as with a constant, the memory is allocated only once when the constant is initialized and stays in memory until it is free'd. Using the constant will point to the memory rather than reallocate it, which can potentionally save memory operations at the cost of constant memory usage.

[Arkshine]

#define PLUGIN "Simple Slots Reservation"
#define VERSION "0.0.6" 

After compilation, same string is replaced everywhere.
Compiler allocates 3 times the same string:

public plugin_init()
{
    register_plugin("Simple Slots Reservation", "0.0.6", "Jhob94")

    register_cvar("Simple Slots Reservation", "0.0.6", FCVAR_SPONLY|FCVAR_SERVER)
    set_cvar_string("Simple Slots Reservation", "0.0.6")
}

VS

new const PLUGIN[] = "Simple Slots Reservation";
new const VERSION[] = "0.0.6";

After compilation, string is allocated one time and this will use references
anywhere string is repeated.

public plugin_init()
{
    register_plugin(referenceString_1, referenceString_2, "Jhob94")

    register_cvar(referenceString_1, referenceString_2, FCVAR_SPONLY|FCVAR_SERVER)
    set_cvar_string(referenceString_1, referenceString_2)
}

Basically, if you have a string which is referenced more than one time, this is good idea to use new const


EDIT: Shit, ninja'd! \o/

Though I'm not sure to agree to

the memory will be allocated (and free'd) each time the string is used, which is a waste of computing power (not that it really matters, though).

Maybe I've misunderstood, but string is allocated one time and is referenced anyway by an address ; the point is just for a same string repeated x times, plugin will allocate x times with a x addresses.

[Jhob94]

Roger that, thanks both

Updated

[Backstabnoob]

Originally Posted by Arkshine Maybe I've misunderstood, but string is allocated one time and is referenced anyway by an address ; the point is just for a same string repeated x times, plugin will allocate x times with a x addresses.

That's good information, thanks.

[Arkshine]

While you are on it, here the french translation with accents:

[fr]
DROPPED_RES = Désolé, un admin vient de prendre sa place réservée, tu as été éjecté du serveur.

[Jhob94]

Originally Posted by Arkshine While you are on it, here the french translation with accents: [fr] DROPPED_RES = Désolé, un admin vient de prendre sa place réservée, tu as été éjecté du serveur.

This is ML from default amxx system. I don't think it's a good idea to change it here, people shouldn't be forced to replace it. I just uploaded for the case someone lost the file somehow.

[Arkshine]

Ah. Then you should not attach it, because if file is changed upstream (which is going to be the case), people will think it's the right version and will replace it. There is no reason to attach it really. Don't bother with it.