[bomnacama]

Helmet Hit Sound

Description:

This is my first plugin, its very simple because im still learning.
This plugin adds an diferent sound to the helmet hit.
Just put the sound in sound/misc.
Credits to joaquimandrade for the help

Please report if something is wrong/buggy.

Originally Posted by devicenull Some examples of plugins that will not be approved: Any plugin that lets you play one sound in response to one event.

Thats what my plugin does, but it can be put in other way. Just think of a plugin that custumizes an existing hit sount

Requires:

hamsandwich
fakemeta

Change-Log:

* 1.0
- First release

[SnoW]

Originally Posted by bomnacama Requires: amxmodx amxmisc hamsandwich fakemeta

Only FM and Ham are modules, so you can just remove the others.

[Exolent[jNr]]

Originally Posted by devicenull Some examples of plugins that will not be approved: Any plugin that lets you play one sound in response to one event.

http://forums.alliedmods.net/newrepl...reply&p=189810

[bomnacama]

ive missed that.every one understands why we cannot post slowhacking, offensive plugins etc.but I cant understant why we cant post a plugin that does that

[ConnorMcLeod]

It's not really a response to an event, + this is a good idea (i haven't listened the sound yet but if it's a helmet sound this is cool).


You don't need to check if the victim is valid, the fact that you register the ham forward with classname "player" is suffisent.
Also, this is not a hudge optimisation, but you can save a var by doing :
get_user_attacker(victim, hitzone, hitzone)

Or you can even try :

get_user_attacker(victim, _, hitzone)
I don't know how the compiler will react as i've never seen native such as :
native get_user_attacker(index,...);

[joaquimandrade]

Originally Posted by ConnorMcLeod It's not really a response to an event, + this is a good idea (i haven't listened the sound yet but if it's a helmet sound this is cool). You don't need to check if the victim is valid, the fact that you register the ham forward with classname "player" is suffisent. Also, this is not a hudge optimisation, but you can save a var by doing : get_user_attacker(victim, hitzone, hitzone) Or you can even try : get_user_attacker(victim, _, hitzone) I don't know how the compiler will react as i've never seen native such as : native get_user_attacker(index,...);

Yes you're right about the classname "player". It is sufficient.
Anyway this is a response to an event. The event is called "being hit in the head when with helmet".

[ConnorMcLeod]

I would consider that a music or a funny sound is a sound in response to an event.
You plugin is more as a nice feature added to the game.

But call that as you want

[joaquimandrade]

Originally Posted by ConnorMcLeod I would consider that a music or a funny sound is a sound in response to an event. You plugin is more as a nice feature added to the game. But call that as you want

This plugin is not mine. Is from bomnacama. I also think that is a nice feature but anyway i think that we can agree that it qualifies as a response to an event. Anyway, if the rule was meant to forbid plugins like this, it is stupid.

PS: Connor, to do this: get_user_attacker(victim, hitzone, hitzone), you must be sure that it is assigned the value to the last parameter after being assigned to the second parameter but, anyway, the code is uglier.

[ConnorMcLeod]

Originally Posted by joaquimandrade you must be sure that it is assigned the value to the last parameter after being assigned to the second parameter

I'm affraid it's not the case

[YamiKaitou]

Are you planning on adding more features to this plugin? If not, then I will unapprove it per the rules