Skip to content

fix: Initialize audioplayer instances sequentially#1941

Merged
spydon merged 5 commits into
bluefireteam:mainfrom
gnarhard:fix/fix_concurrent_audio_instance_load_error
Sep 3, 2025
Merged

fix: Initialize audioplayer instances sequentially#1941
spydon merged 5 commits into
bluefireteam:mainfrom
gnarhard:fix/fix_concurrent_audio_instance_load_error

Conversation

@gnarhard

Copy link
Copy Markdown
Contributor

Description

When creating lots of audio pools, an "Unknown Error" can happen if too many players are loaded concurrently. By breaking the instantiation of new players into a serially-loaded structure, we can load 100s of players and effectively remove this edge case.

Checklist

  • The title of my PR starts with a Conventional Commit prefix (fix:, feat:, refactor:,
    docs:, chore:, test:, ci: etc).
  • I have read the Contributor Guide and followed the process outlined for submitting PRs.
  • I have updated/added tests for ALL new/updated/fixed functionality.
  • I have updated/added relevant documentation and added dartdoc comments with ///, where necessary.
  • I have updated/added relevant examples in example.

Breaking Change

  • Yes, this is a breaking change.
  • No, this is not a breaking change.

Related Issues

@gnarhard

Copy link
Copy Markdown
Contributor Author

Looks like some doc code from #1928 got pulled into this. If you wanna close that PR, we can continue working in here.

@spydon spydon changed the title Fix concurrent audio instance load error fix: Initialize audioplayer instances sequentially Aug 26, 2025
@spydon spydon merged commit 663fff2 into bluefireteam:main Sep 3, 2025
26 checks passed
@gnarhard gnarhard deleted the fix/fix_concurrent_audio_instance_load_error branch October 2, 2025 03:23
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.

2 participants