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

mod: construction sound no longer loops, fixes #2429 #2484

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

nanocirs
Copy link

Solved the issue where construction sound looped every 4s. Construction sound is now only triggered when starting to build.

Copy link

sonarcloud bot commented Feb 20, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

@Vorschreibung
Copy link

Vorschreibung commented May 25, 2024

Heya, I'm not a regular contrib here, so take my notes with a grain of salt - tried your patch and can confirm that this works. Plays sound only once the construction starts.

Even if you die and come back, as long as it is still partially constructed, will also not play the sound again until it's either destroyed or construction aborted due to idleness.

Suggest to add a comment above the conditional that mentions that constructible->s.angles2[0] here is (AFAIS) the health of the constructible (grep for addhealth) - and/or to mention something like // play construction sound only at the start of construction.

I have checked with 2.60b and it seems to behave the exact same way now. 🤘 😊

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.

None yet

2 participants