Skip to content

Increase sound source pool size to the maximum#21174

Merged
abcdefg30 merged 1 commit into
OpenRA:bleedfrom
PunkPun:fix-sound
Oct 30, 2023
Merged

Increase sound source pool size to the maximum#21174
abcdefg30 merged 1 commit into
OpenRA:bleedfrom
PunkPun:fix-sound

Conversation

@PunkPun

@PunkPun PunkPun commented Oct 30, 2023

Copy link
Copy Markdown
Member

Closes #19724

kcat/openal-soft#580
https://github.com/kcat/openal-soft/blob/b6aa73b26004afe63d83097f2f91ecda9bc25cb9/alc/alc.cpp#L1468-L1485
https://github.com/kcat/openal-soft/blob/b6aa73b26004afe63d83097f2f91ecda9bc25cb9/alc/alc.cpp#L3191-L3203

If I try to create a 257th source it errors with

[ALSOFT] (WW) Error generated on context 0x7fd37b9509d0, code 0xa005, "Exceeding 256 source limit (256 + 1)"

with the AL error being

Out of Memory

Comment thread OpenRA.Platforms.Default/OpenAlSoundEngine.cs Outdated
@PunkPun PunkPun force-pushed the fix-sound branch 2 times, most recently from 3649675 to 91381c3 Compare October 30, 2023 15:58
Comment thread OpenRA.Platforms.Default/OpenAlSoundEngine.cs Outdated
@PunkPun PunkPun force-pushed the fix-sound branch 2 times, most recently from c0e73c7 to 670ec03 Compare October 30, 2023 16:15
@PunkPun PunkPun added this to the Next Release milestone Oct 30, 2023
@PunkPun PunkPun changed the title Add a dynamic sound source amount Increase sound source pool size to the maximum Oct 30, 2023
@abcdefg30 abcdefg30 merged commit c0da41a into OpenRA:bleed Oct 30, 2023
@abcdefg30

Copy link
Copy Markdown
Member

Changelog

@PunkPun PunkPun deleted the fix-sound branch October 31, 2023 09:34
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.

Airstrike Notification/Sound Priority System

3 participants