Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

move player pain sound to the sg_oof sound group #2092

Merged
merged 1 commit into from
Dec 15, 2024
Merged

Conversation

fabiangreffrath
Copy link
Owner

Prevents the chainsaw idle sound and the player pain sound from interruting each other.

https://www.doomworld.com/forum/topic/112333-this-is-woof-1501-dec-06-2024/?page=103&tab=comments#comment-2877815

This fix seems plausible since both "Oof!" and "Ouch!" are voiceful expressions. ;)

Prevents the chainsaw idle sound and the player pain sound from interruting each other.

https://www.doomworld.com/forum/topic/112333-this-is-woof-1501-dec-06-2024/?page=103&tab=comments#comment-2877815

This fix seems plausible since both "Oof!" and "Ouch!" are voiceful expressions. ;)
@ceski-1
Copy link
Collaborator

ceski-1 commented Dec 14, 2024

This is normal behavior, right? What are the side effects to changing this? Are other sounds now being cut off?

@fabiangreffrath
Copy link
Owner Author

This is normal behavior, right? What are the side effects to changing this? Are other sounds now being cut off?

Now "Ouch!" and "Oof!" could interrupt each other.

@fabiangreffrath fabiangreffrath merged commit 0c5d0f5 into master Dec 15, 2024
8 checks passed
@fabiangreffrath fabiangreffrath deleted the plpain2 branch December 15, 2024 09:33
@fabiangreffrath
Copy link
Owner Author

The alternative solution would have been to move the saw sounds into their own group, but I think this change is less disruptive and follows the existing logic.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants