Hit Sounds in the Core

[Austin]

Damaged sound

Currently when a mobile is damaged the core plays a sound effect.

Code: Select all

do_imhit_effects()
{
        if (gender == GENDER_MALE)
		play_sound_effect( this, SOUND_EFFECT_MALE_DEFENSE );
	else
		play_sound_effect( this, SOUND_EFFECT_FEMALE_DEFENSE );
     ...
}

Vote on the poll... ill let it run for 4 days.

[ncrsn]

I'd like to know reasons behind the votes casted. I, myself, voted "should be removed from the core entirely", because I like to be able to control how things work out: if stuff is coded in the core, modifying stuff tends to require heavy packethooking. Why not lend a hand to advanced users and give others an another reason to use core + distro?

[*Edwards]

Honestly, I would handle the DamagedSound by myself. Well, that's for my part.

[OldnGrey]

I am still a fan of the idea of optionally enabling something to be handled by script. A good example for me is decay.

I guess it just lessens the impact of a core change whilst allowing a scripter to deal with it when they are ready.

[Yukiko]

I am in favour of having the ability to handle as much scriptside as possible.

I vote with OldNGrey on this one. It is nice to have the option.

I'd say default to core handling it unless the setting, I assume it would be in serverspecopts or whatever that file is named, is set to hand it off to scripts.

Now if having an option adds much processing overhead to the server then let core pass it to scripts with no option.

[ncrsn]

I am still a fan of the idea of optionally enabling something to be handled by script.

Well okay, I could live with that.

-- edit:

Way to go Austin!