-
-
Notifications
You must be signed in to change notification settings - Fork 672
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
Crash in ios #34
Comments
It could be that if (!_eventSink) return; I also have a lot to learn about iOS memory management still, so maybe it's something even more crazy like the self instance being free'd. But I'd start with the above guard first. In any case, I am going off to sleep now. You are of course welcome to try the above code suggestion and see if it works for you in the meantime, assuming you are able to reproduce the issue. |
Thank you! When the plugin is updated I'll use it and redeploy, and see if that stops this crash. |
I've added the above line of code to the latest git commit if you'd like to test it. |
Thanks! App works like a charm now. Would recommend releasing as fast as possible as this is quite the critical error on iOS |
Great! I was waiting on confirmation on some other iOS code that will also be included in the release, but it seems that things are working well enough that you haven't noticed any issues so I've just published a new release. If you have some spare time, I would also appreciate your feedback on #35 . |
It's been 24 hours, and I'm not seeing any crashes now that I'm using the latest commit. One interesting bit of context: One beta tester showed me that the crash happened when
|
This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs, or use StackOverflow if you need help with just_audio. |
I'm seeing a crash in crashalytics. (I'm also using audio_service)
I notice that the free RAM looks limited. Do you think this is just because my app is currently RAM heavy?
Model: iPhone 6s
Version: 13.3.0 (17C54)
RAM free: 58.27 MB
The text was updated successfully, but these errors were encountered: