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

Custom campaign sound #3676

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from
Draft

Custom campaign sound #3676

wants to merge 2 commits into from

Conversation

xtremeqg
Copy link
Contributor

Unified sound banks.

SoundBankIDs have been completely removed. Sound samples are now referenced only by SoundSmplID.
The first 1160 SoundSmplIDs are reserved for sound effects and mentor speech, in that order.
It is now possible to play samples by name, get_sample_id() provides the SoundSmplID of any named sample.
Lots of magic numbers have been replaced with enums.
Types of several struct fields have been corrected.
Console commands "sound.test" and "speech.test" have been replaced with "sample.test".
Campaign sounds are accessible starting from sample ID 1661.
Samples can also be specified by name in [sounds] crtr block.

SoundBankIDs have been completely removed.

Sound samples are now referenced only by SoundSmplID.

The first 1160 SoundSmplIDs are reserved for
sound effects and mentor speech, in that order.

It is now possible to play samples by name, get_sample_id()
provides the SoundSmplID of any named sample.

Lots of magic numbers have been replaced with enums.

Types of several struct fields have been corrected.

Console commands "sound.test" and "speech.test"
have been replaced with "sample.test".
Campaign sounds are accessible starting from sample ID 1661.

Samples can also be specified by name in [sounds] crtr block.
@Loobinex Loobinex marked this pull request as draft November 26, 2024 01:17
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.

1 participant