[vilaemail]

Originally Posted by Nextra Wait for an AMXModX plugin approver to review your plugin. This can take some time depending on how busy these people are.

No problem, I have patience and understand them.. There are a lot of pending plugins

[ditmesteam]

I see one bug: when I beted in round 7 (for example) message of result bet do not display,in round 8 I can see message!

[vilaemail]

Originally Posted by ditmesteam I see one bug: when I beted in round 7 (for example) message of result bet do not display,in round 8 I can see message!

So you bet. Round ends. And only when new round starts you can see message that you won? That's strange since payoff is done immediately after round end. Maybe you are running some other plugins that hooks on same events I have hooked (terrorist win and ct win sound). Please post your plugins.ini

[ditmesteam]

You say right, I will send feedback later!

[vilaemail]

Script upgraded to v1.7. Many new features!

Originally Posted by vilaemail Change log: v1.7 Made bet_addsenabled and bet_mustbedead cvars. Made amx_advertisebet and say /advertisebet commands Updated cvar handling Fixed say clcmd registration (flags where 0. Changed to ADMIN_USER ) Added Russian and Chinese translation v1.6 Reorganized code Fixed most of the compiler warnings (tag mismatch) Updated translations (sk, cz) v1.5a Changed g_messageTable to const Fixed minor printing bug v1.5 Added cvar bet_chatenabled (def. 1) v1.4 Reordered code Optimized code Optimized memory usage Removed Russian translation v1.3b Improved code performance. Removed some unnecessary code. v1.3 and v1.3a Improved code performance. Made code more consistent. Fixed minor bug. v1.2 Updated translations (ru, de). Improved code performance. v1.1 Fixed minor typing mistake. Updated translations (nl).

[Nextra]

Good Job, some suggestions:
- There is no need to add another function call through stocks. If you want to use simpler terms for get_pcvar_num use defines:

PHP Code:

#define GetChatEnabled() bool:get_pcvar_num(cvar_chatEnabled)
#define GetAddsEnabled() bool:get_pcvar_num(cvar_addsEnabled)
#define GetAliveEnabled() bool:get_pcvar_num(cvar_aliveEnabled) 


- You can omit the int: tag since thats how every var will work when not declared another tag (neglecting the fact that pawn is typeless anyway).
- Ad with only one 'd'. Add is something else and may confuse people.

[matsi]

This i don't understand:

Originally Posted by Nextra PHP Code:     for (i=0; i<playerCount; i++)     {         //player = Players[i]  - no need for this         //team = cs_get_user_team(Players[i]) - no need for this either         switch(cs_get_user_team(Players[i]))         {             case CS_TEAM_T:             {                 aliveT++             }             case CS_TEAM_CT:             {                 aliveCT++             }         }     }

[Nextra]

What exactly is the thing you do not understand?

You do have knowledge of for loops and switch statements, do you?

[YamiKaitou]

Originally Posted by matsi As for loop you should avoid re-indexing arrays.

There is no re-indexing in the snippet.

And for cs_get_user_team i would use: new CsTeams:team = cs_get_user_team( index )

What is the point if you are only going to use the return value once?

EDIT: And you have blanked it for the second time as I was responding, screw it now

[vilaemail]

Originally Posted by Nextra Good Job, some suggestions: - There is no need to add another function call through stocks. If you want to use simpler terms for get_pcvar_num use defines: PHP Code: #define GetChatEnabled() bool:get_pcvar_num(cvar_chatEnabled) #define GetAddsEnabled() bool:get_pcvar_num(cvar_addsEnabled) #define GetAliveEnabled() bool:get_pcvar_num(cvar_aliveEnabled)  - You can omit the int: tag since thats how every var will work when not declared another tag (neglecting the fact that pawn is typeless anyway). - Ad with only one 'd'. Add is something else and may confuse people.

-I was guided by this:

new g_enabled = register_cvar("csdm_enabled", "1") //OR new g_enabled = get_cvar_pointer("csdm_enabled") stock SetCSDM(num) set_pcvar_num(g_enabled, num) stock GetCSDM() return get_pcvar_num(g_enabled)

I optimized them to defines.
-Int tag is there because of compiler warnings, which are very annoying.
-Ad misspell fixed.