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

Catch errors in ExoPlayerAudioPipeline #3716

Merged

Conversation

nielsvanvelzen
Copy link
Member

In some cases initializing an audio effect in Android can fail. I was not able to find good documentation on this so this fix will fallback to NO-OP for now when initialization fails.

Changes

  • Catch errors in ExoPlayerAudioPipeline

Issues

Crash on some Android devices reported via Matrix.

@nielsvanvelzen nielsvanvelzen added bug Something isn't working crash Bug causing app crashes labels Jul 3, 2024
@nielsvanvelzen nielsvanvelzen added this to the v0.17.0 milestone Jul 3, 2024
@nielsvanvelzen nielsvanvelzen merged commit 9ead8dd into jellyfin:master Jul 4, 2024
5 checks passed
@nielsvanvelzen nielsvanvelzen deleted the audio-pipeline-crash branch July 4, 2024 12:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working crash Bug causing app crashes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants