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 incorrect duration value on android that comes from an audio stream #2567

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

mrEuler
Copy link
Contributor

@mrEuler mrEuler commented Aug 5, 2024

I didn't have this issue, when there was no audio playback in SkVideo. Now in some videos, the duration is totally incorrect (for example instead of 210'000 ms, it could be 78'123 ms, so no logic or dependency). That caused the issue of video interruption, that was caused, when the current playback time was larger than a "fake duration".
Fixed this locally and tested a lot, works fine.

Issue was (is) only on Android.

@mrEuler
Copy link
Contributor Author

mrEuler commented Aug 5, 2024

@wcandillon I was talking about this one ^^

@wcandillon wcandillon self-requested a review August 6, 2024 05:24
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