[ConnorMcLeod]

Damn, again one of these
Multi-messages + ML would be better for such a plugin.

[Flasher]

Originally Posted by ConnorMcLeod Damn, again one of these

Originally Posted by ConnorMcLeod Multi-messages + ML would be better for such a plugin.

Multi-messages = several messages for one event picked by random?
ML: Sounds are in english. So messages too. If you have translated sounds no one stops you from modifying ini file to match your language. If I will make it ML then I have to use ML tags in ini file and have matching ML file. This makes not easy modifying announcements levels. So it cuts both ways.

[Arkshine]

- Yes
- It doesn't matter really if the sound don't match the messages, it's up to the admin to change the sounds. But when you have client text message, ML should be always used.

[Flasher]

- May be later.
- Also may be later.
Time is a gold. So everyone decide how to spend it.

[Arkshine]

About your code, just minors things :

- You hook say and say_team, I would hook the command with register_clcmd(), it will avoid to call 2 natives each time someone types something. Letting the engine doing the check is better.
- equali(temp, "0"), you can do directly : temp[0] == '0'
- About the hook of player's spawn, use Ham_spawn to hook properly. There is a tuto about that in the tutorials section. ( page 4 or 5 )
- The loop using MAX_PLAYERS is not right. The right value is from get_maxplayers() ( max slot of server ). Use a global var.
- To avoid unnecessary natives calls, I would use get_players() to get a list of connected player.

[Flasher]

Originally Posted by Arkshine About your code, just minors things : - You hook say and say_team, I would hook the command with register_clcmd(), it will avoid to call 2 natives each time someone types something. Letting the engine doing the check is better. - equali(temp, "0"), you can do directly : temp[0] == '0' - About the hook of player's spawn, use Ham_spawn to hook properly. There is a tuto about that in the tutorials section. ( page 4 or 5 ) - The loop using MAX_PLAYERS is not right. The right value is from get_maxplayers() ( max slot of server ). Use a global var. - To avoid unnecessary natives calls, I would use get_players() to get a list of connected player.

Thanks for your time!
- I tried register_clcmd(). It works for "say /utsounds", but not for "say /utsounds on". So I have to use say and say_team. This is not so terrible I think.
- temp[0] == '0'. Ok.
- Ham_spawn - I will give it a look.
- About players - I get your point.

[BOYSplayCS]

Ham_Spawn() will preform your hook much faster and more efficiently.

About your register_clcmd() issue, this is probably because of the third space for the word "on".

[Flasher]

I tried
RegisterHam(Ham_Spawn, "player", "playerSpawn", 1);
but is_user_alive(id) is 0 at playerSpawn(id).
Mod - pure HLDM.

register_clcmd() - I did this such:
register_clcmd("say /utsounds", "cmdInfo");
register_clcmd("say /utsounds on", "cmdOn");
register_clcmd("say /utsounds off", "cmdOff");

And only cmdInfo was triggered (on "say /utsounds on" nothing was triggered).
May be this is because HL client convert say command to this:
say "/utsounds on"
?

[Hawk552]

Originally Posted by Arkshine [...] ML should be always used.

omg

[Contrenature]

Nothing special, old sounds.
We have much better sounds effects, if some interesning, send me PM!