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 typo jlong -> jdouble for seek method of the Video (android only) #2532

Merged
merged 1 commit into from
Jul 25, 2024

Conversation

mrEuler
Copy link
Contributor

@mrEuler mrEuler commented Jul 12, 2024

fixes #2531

In java native part, the seek method got double parameter instead of long, but in cpp part, the casting wan't changed, what caused the issue of incorrect value.

@mrEuler
Copy link
Contributor Author

mrEuler commented Jul 12, 2024

@wcandillon let's merge this and create a next version, as this is the critical fix.

@mrEuler
Copy link
Contributor Author

mrEuler commented Jul 12, 2024

signed the Contributor License Agreement (CLA) just in case

@wcandillon wcandillon self-requested a review July 25, 2024 04:27
@wcandillon
Copy link
Contributor

thanks a lot :)

@wcandillon wcandillon merged commit 76343ce into Shopify:main Jul 25, 2024
8 of 10 checks passed
Copy link
Contributor

🎉 This issue has been resolved in version 1.3.9 🎉

The release is available on:

Your semantic-release bot 📦🚀

@mrEuler
Copy link
Contributor Author

mrEuler commented Aug 5, 2024

@wcandillon thank YOU! also found one more issue, making a PR :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Video seek (Android) issue
2 participants