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
Fixed memory leak #222
Fixed memory leak #222
Conversation
Maybe I need to put this free's somewhere else. I guess there can be cases this fix doesn't account for, but where I put them now fixed the problem I'm having. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Move freeing default_capture_id
to another conditional branch.
Co-authored-by: TimTim <i@timtim.hk>
Co-authored-by: TimTim <i@timtim.hk>
thank you, @wegylexy! :-) |
@andrewrk Could we soon merge this fix? |
@andrewrk Do you also have time to evaluate the other pull requests? Especially those WASAPI ones as the current thing is partially broken on Windows 10 v2004. |
No I won't be making another release for about 2 years, but then it should pick back up. The biggest problem with pull requests is testing. QA for this software is near impossible, we need actual hardware and half the time I ask people if they tested their code, they say "no, I just guessed this would work". |
Fixed memory leak (andrewrk#222)
First of all, thanks for this amazing lib. The API looks amazing!
So I just started testing soundio, but I kept getting a visual leak detection. The leak was caused by
soundio_connect(soundio);