-
Notifications
You must be signed in to change notification settings - Fork 23
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
Please add ReleaseFrame func like Gav1Decoder #39
Comments
@FongMi Could you explain what you mean and why this function is important |
My player interface can adjust the ExtensionRendererMode at any time. As long as it is changed from EXTENSION_RENDERER_MODE_PREFER to EXTENSION_RENDERER_MODE_ON, an error will occur. Using DefaultRenderersFactory it won't happen. So I think it is when the player has been released, but ffmpeg did not receive the notification to release, causing the decoder error. |
@FongMi It's the issue with surface as mentioned in google/ExoPlayer#9459 |
I see. |
Hey @FongMi. Just wanted to let you know that I kinda solved this issue by recreating the surface everytime video switches decoder. Implementation is in this PR:- anilbeesetti/nextplayer#785 |
Thanks, I will test again. |
@anilbeesetti have u try to use TextureView ? Screen will not update, and keep same frame. |
I have tested this pr, it does not need to recreate the surface. |
The text was updated successfully, but these errors were encountered: