Skip to content

replace ALooper_pollAll (removed in API 34) #38

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

tomas-mraz
Copy link
Contributor

The new NDK does not want to compile because of the usage of ALooper_pollAll.
The recommended action is to replace with ALooper_pollOnce.

@xlab
Copy link
Owner

xlab commented Jul 5, 2025

Thanks! I am also glad this is still useful for some :)

@xlab
Copy link
Owner

xlab commented Jul 5, 2025

However that android.go file was autogenerated. Shouldn't be changed by hand, the original NDK has to be updated instead and re-generate the bindings.

Latest LTS for NDK is r27c or 27.2.12479018 (https://developer.android.com/ndk/downloads)

Similar issue ocornut/imgui#8013

I agree that bumping NDK is a bit more overhead than changing a single func call, but it seems like a correct path to move forward with bindings maintenance, since there might be other deprecations.

I'll keep PR open for now. Anyone can find it just in case.

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