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

Initial SDL audio implementation #12

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

ccawley2011
Copy link
Member

This is based on the threaded OSS implementation, and it does work, however the audio breaks up frequently and I don't fully understand how the sound system in ArcEm works, so I'd appreciate any advice on how to improve things.

@Phlamethrower
Copy link
Contributor

Sorry for taking a while to take a look at this.

I've been having a look at it over the past few days. I've found a couple of bugs in the main sound code which are stopping things from working properly, but really the main issue is that the threaded version of the OSS Sound_HostBuffered wasn't very good to begin with. It was detecting false underflow & overflow conditions, and was constantly oscillating between high & low buffer utilisation instead of settling at the desired place. So I've been working on a new version of the SDL code that should behave better (and should hopefully be a better base for other ports to use), but it's taking a while to get it to a state where it should work properly in most/all situations.

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