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

Android) Use ALooper_pollOnce #659

Closed
wants to merge 1 commit into from
Closed

Conversation

hsdk123
Copy link
Contributor

@hsdk123 hsdk123 commented Jan 19, 2025

Due to error when trying to build:

/Users/hkim809/Desktop/SoulEngineProject/Light.vn-core/_deps/magnum/09ec007dcdf254550026d6b8f1a7dfd74874e1e9/src/Magnum/Platform/AndroidApplication.cpp:672:24: error: 'ALooper_pollAll' is unavailable: obsoleted in Android 1 - ALooper_pollAll may ignore wakes. Use ALooper_pollOnce instead. See The API documentation for more information
  672 |         while((ident = ALooper_pollAll(
      |                        ^
/Users/hkim809/Library/Android/sdk/ndk/28.0.12916984/toolchains/llvm/prebuilt/darwin-x86_64/bin/../sysroot/usr/include/android/looper.h:234:5: note: 'ALooper_pollAll' has been explicitly marked unavailable here
  234 | int ALooper_pollAll(int timeoutMillis, int* outFd, int* outEvents, void** outData)
      |     ^
1 error generated.

@hsdk123
Copy link
Contributor Author

hsdk123 commented Jan 24, 2025

@mosra were you able to look through this?

@mosra mosra added this to the 2025.0a milestone Jan 25, 2025
@mosra
Copy link
Owner

mosra commented Jan 25, 2025

Not yet, sorry. I have a giant change related to image representation in this repository that I want to finish first. Hopefully that happens today or tomorrow, and then I can look.

@hsdk123
Copy link
Contributor Author

hsdk123 commented Feb 16, 2025

@mosra have you been able to take a look through?
This is a blocker for raising the ndk version and keeps me stuck on an older version of c++.

@mosra
Copy link
Owner

mosra commented Feb 18, 2025

Commited as 2219abd with some minor formatting changes. Sorry it took so long, I had too many changes happening at once and had to finish them all first. I'll try to merge #660 at least partially as soon as I have another chance.

@mosra mosra closed this Feb 18, 2025
@hsdk123
Copy link
Contributor Author

hsdk123 commented Feb 18, 2025

No worries, thanks a bunch as always!

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

Successfully merging this pull request may close these issues.

2 participants