Skip to content

Conversation

@mdydek
Copy link
Contributor

@mdydek mdydek commented Oct 23, 2025

Closes RNAA-222

⚠️ Breaking changes ⚠️

Introduced changes

  • logic of buffering had some issues, when streaming thread was reaching upper bound of buffer, some artefacts were incorporated in signal, this approach refactors this case by introducing conditional variable for synchronization between threads

Checklist

  • Linked relevant issue
  • Updated relevant documentation
  • Added/Conducted relevant tests
  • Performed self-review of the code
  • Updated Web Audio API coverage
  • Added support for web
@mdydek
Copy link
Contributor Author

mdydek commented Oct 23, 2025

you can hear a difference by using link provided in example, compare main version to this branch

@mdydek mdydek requested a review from poneciak57 October 28, 2025 11:33
@mdydek mdydek self-assigned this Oct 28, 2025
@mdydek mdydek merged commit 0db0ddf into main Oct 29, 2025
3 checks passed
@mdydek mdydek deleted the refactor/streamer branch October 29, 2025 16:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

4 participants