Opened 9 years ago

Closed 8 years ago

Last modified 8 years ago

#3486 closed defect (fixed)

[PATCH] Sheeps keep making sounds after death

Reported by: Stan Owned by: Stan
Priority: Should Have Milestone: Alpha 20
Component: UI & Simulation Keywords: patch
Cc: Patch:

Description

Some players reported that the sheep continues making sound after dying this is because line 552 of templateLoader.cpp the sound component is transfered to the dead unit.

So here is a one liner that fixes that.

Attachments (2)

#3486.diff (512 bytes ) - added by Stan 9 years ago.
Remove the sound component from dead objects.
#3486.2.diff (597 bytes ) - added by Stan 9 years ago.
Add a comment.

Download all attachments as: .zip

Change History (9)

comment:1 by Stan, 9 years ago

Owner: set to Stan
Status: newassigned

by Stan, 9 years ago

Attachment: #3486.diff added

Remove the sound component from dead objects.

by Stan, 9 years ago

Attachment: #3486.2.diff added

Add a comment.

comment:2 by historic_bruno, 9 years ago

You mean sound when it's selected? I don't think that's a big deal, just like the unit portrait shows a living sheep, the sound is simply a way to identify it.

comment:3 by Stan, 9 years ago

Implodedok asked me to fix it :) So I submitted a patch :)

comment:4 by elexis, 8 years ago

To me it makes more sense to disable the sounds of animal corpses. After all it's not the portrait but the unit on the battlefield that emits the sound. So I'd be in favor of fixing it rather than closing it as won't fix.

The patch should fix this for all animal types though (for example chicken and camels too).

comment:5 by elexis, 8 years ago

Resolution: fixed
Status: assignedclosed

In 17399:

Don't emit sounds when selecting dead animals. Patch by Stan. Fixes #3486.

comment:6 by elexis, 8 years ago

Component: Music & Sound FXUI & Simulation
Keywords: review removed

Thanks for the patch

comment:7 by elexis, 8 years ago

In 17400:

Don't comment out code. Refs #3486.

Note: See TracTickets for help on using tickets.