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

Muffle sounds when the player is under water #140

Merged

Conversation

hemebond
Copy link
Contributor

@hemebond hemebond commented Mar 4, 2024

daftmugi ported Ironwail's audio acoustic feature, muffling the intensity of sounds when the player is underwater, to the wrath-darkplaces repo. I've cherry-picked the commit (daftmugi/wrath-darkplaces@d732247).

@hemebond hemebond changed the title Muffle sounds when the player is underwater Muffle sounds when the player is under water Mar 4, 2024
daftmugi and others added 3 commits March 11, 2024 01:43
Enabled by default. Use 'snd_waterfx 0' to disable it.

Adapted code from Ironwail so that it works in this engine.

Cherry-picked from daftmugi/wrath-darkplaces
@bones-was-here
Copy link
Contributor

Nice effect. I made some adjustments:

Rebased on current master.

Moved 1 change into the original commit: qboolean -> qbool. This allows that commit to appear in the master history without it being a pain when using git bisect.

Squashed your fixes into a single commit, and omitted a few of the newlines you added there (the ones between the UNDERWATER EFFECT header and the code for that feature). Generally in DP extra newlines only go between sections, not within a section.

Added a commit simplifying the code and clarifying that cvar snd_waterfx is more than just an on/off switch.

Going to merge.

@bones-was-here bones-was-here merged commit cf08534 into DarkPlacesEngine:master Mar 10, 2024
1 check passed
@hemebond hemebond deleted the feature/underwater-acoustics branch March 11, 2024 00:28
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