-
Notifications
You must be signed in to change notification settings - Fork 255
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
[question/might be a BUG] AAudio.h:809:9: error: unknown type name 'bool' in NDK 21d #1281
Comments
i note that although many of our header files use <math.h> is a false positive (it mentions
(the + lines are files that include <stdbool.h> but don't use bool. of course we have some of them too.) |
(just to be explicit: your short-term workaround is |
That worked well, at least as a workaround for now. |
ndk_binder: https://android-review.googlesource.com/c/platform/frameworks/native/+/1331456 camera & media: https://android-review.googlesource.com/c/platform/frameworks/av/+/1333894 nnapi: https://android-review.googlesource.com/c/platform/frameworks/ml/+/1333955 the audio API isn't in AOSP yet, so the CL for that is internal: https://googleplex-android-review.git.corp.google.com/c/platform/frameworks/av/+/11851227 |
Thank you very much for the prompt reaction. Much appreciated! |
AAudio is in AOSP at: |
aye, but the specific new API that uses bool isn't in AOSP yet :-) |
Bug: android/ndk#1281 Test: treehugger Change-Id: Ibba9b1b03e92da8a664145a269732c638b2ec550
...to files with C API using bool. (I've left the ones where the only uses of bool are in C++ anyway, since those headers are obviously not usable from C.) Bug: android/ndk#1281 Test: treehugger Change-Id: I616a8482bcdbb391ec8c028d6b6a70b1d9d08960
I think the patches that fix this are in now so I just need to merge them into the r21 sysroot and make the update. |
...to files with C API using bool. (I've left the ones where the only uses of bool are in C++ anyway, since those headers are obviously not usable from C.) Bug: android/ndk#1281 Test: treehugger Exempt-From-Owner-Approval: Cherry pick to release branch Change-Id: I616a8482bcdbb391ec8c028d6b6a70b1d9d08960 (cherry picked from commit 09f4184)
Has this been closed? I am hitting it with NDK 21.3.6528147. |
I see that stdbool.h was added to AAudio.h in this CL on June 12, 2020: |
Because that's too old: https://github.com/android/ndk/wiki/Changelog-r21#r21e You're using r21d. |
Since NDK failed to do that: android/ndk#1281 Signed-off-by: Zhao Zhili <zhilizhao@tencent.com>
Bug: android/ndk#1281 Test: treehugger Change-Id: Ibba9b1b03e92da8a664145a269732c638b2ec550 (cherry picked from commit 9dafdd6)
Bug: android/ndk#1281 Test: treehugger Change-Id: Idd009254e41b8b5136bc4d2d9a49c130780f98d5 (cherry picked from commit 9c1fca2)
...to files with C API using bool. (I've left the ones where the only uses of bool are in C++ anyway, since those headers are obviously not usable from C.) Bug: android/ndk#1281 Test: treehugger Change-Id: I616a8482bcdbb391ec8c028d6b6a70b1d9d08960 (cherry picked from commit 09f4184)
Since NDK 21d (21.3.6528147), I'm getting the following compiler error:
.....\21.3.6528147\toolchains\llvm\prebuilt\windows-x86_64\sysroot\usr\include\aaudio\AAudio.h:809:9: error: unknown type name 'bool'
The same setup worked well with NDK 21c (21.2.6472646) and below.
Any help would be welcome.
Thanks in advance
The text was updated successfully, but these errors were encountered: