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

Update fnc_doCallout #355

Merged
merged 2 commits into from
Feb 18, 2024
Merged

Conversation

rekterakathom
Copy link
Contributor

@rekterakathom rekterakathom commented Jul 2, 2023

When merged this pull request will: Improve fnc_doCallout

  1. Describe what this pull request will do

Remove some obsolete code and makes sound originate roughly from the unit's face. Improves caching performance and makes the callout a bit more realistic with zero performance penalty.

Edit: The leading slash can be removed because since Arma 3 v2.10 playSound3D simply ignores the slash so it doesn't need to be manually removed. Source: Biki

  1. Each change in a separate line
  • Remove obsolete code (since Arma 3 v2.10) from caching logic

  • Make callout sound originate from unit's eyePos instead of their position

Since Arma 3 v2.10, leading slashes are allowed for playSound3D, so no need to bother removing it.
The callout is now played from the unit's eyePos instead of their position.
@rekterakathom rekterakathom changed the title Improve fnc_doCallout Update fnc_doCallout Jul 2, 2023
@jokoho48 jokoho48 added this to the 2.6.1 milestone Oct 4, 2023
@nk3nny nk3nny merged commit 8d573ae into nk3nny:master Feb 18, 2024
@nk3nny nk3nny requested review from jokoho48 and nk3nny February 18, 2024 12:55
rekterakathom added a commit to rekterakathom/LambsDanger that referenced this pull request May 28, 2024
Reverts a part of nk3nny#355 because of recent discovery that files included in addons must not have a leading slash
jokoho48 pushed a commit that referenced this pull request Oct 14, 2024
Reverts a part of #355 because of recent discovery that files included in addons must not have a leading slash
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants