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

OpenAL: Don't request samples if buffers are full #4271

Merged
merged 2 commits into from Oct 3, 2016

Conversation

ligfx
Copy link
Contributor

@ligfx ligfx commented Oct 2, 2016

Makes the buffering code a bit more explicit (circular buffer, but blocks until full buffers get unqueued by OpenAL), and fixes a bug in the startup of Super Mario Sunshine: https://bugs.dolphin-emu.org/issues/9811

Broken into two commits for ease of reading the diffs.


This change is Reviewable

@ligfx ligfx changed the title OpenAL: Don't request samples if buffers aren't free OpenAL: Don't request samples if buffers are full Oct 2, 2016
ALint iBuffersFilled = 0;
ALint iBuffersProcessed = 0;
unsigned int nextBuffer = 0;
unsigned numBuffersQueued = 0;

This comment was marked as off-topic.

This comment was marked as off-topic.

Makes the buffering code a bit more explicit (circular buffer, but
blocks until individual buffers get unqueued by OpenAL), and fixes a
bug in the startup of Super Mario Sunshine:
https://bugs.dolphin-emu.org/issues/9811
@shuffle2
Copy link
Contributor

shuffle2 commented Oct 3, 2016

@skidau ?

@shuffle2
Copy link
Contributor

shuffle2 commented Oct 3, 2016

@dolphin-emu-bot rebuild

@shuffle2 shuffle2 merged commit 7f41066 into dolphin-emu:master Oct 3, 2016
@ligfx ligfx deleted the audiofix branch May 24, 2017 21:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
3 participants