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

Fix spinning cursor on OS X #1

Merged
merged 3 commits into from
Jan 6, 2015
Merged

Fix spinning cursor on OS X #1

merged 3 commits into from
Jan 6, 2015

Conversation

slimsag
Copy link
Contributor

@slimsag slimsag commented Jan 6, 2015

  • Fix the spinning cursor / UI deadlock on OS X.
  • Make less memory allocations when updating the texture / video display.
  • A better OpenAL usage (no flushing, better buffering on channels, and a re syncing pattern).

This does not stop the clicking issue entirely.

@slimsag
Copy link
Contributor Author

slimsag commented Jan 6, 2015

I am going to bed now -- but I will send another PR sometime tomorrow (if you are ok with the above changes) from my apudebug branch.

I researched further into the cause of clicking, and to completely rule out Azul3D I did so against the sdl backend. I found that, at least on my system, the APU only sends on average 34khz audio over the audio.Input() channel.

Obviously, you can't playback 34khz audio at a rate of 44.1khz 😞. I think it's likely to be a bug in your code somewhere, but I don't know enough about NES emulation to know for sure.

Details at: slimsag@be7afa1

P.S. The timestamps on my commits are way off, something's up with my system clock =)

nwidger added a commit that referenced this pull request Jan 6, 2015
Fix spinning cursor on OS X
@nwidger nwidger merged commit 6a463fc into nwidger:master Jan 6, 2015
nwidger added a commit that referenced this pull request Jan 14, 2015
Fix spinning cursor on OS X
nwidger added a commit that referenced this pull request Jan 14, 2015
Fix spinning cursor on OS X
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