Age | Commit message (Collapse) | Author |
|
Change-Id: I6a65efc1be792cea73686a461cfc0b0c517d0447
|
|
Primitive type fields are auto-initialized with '\0' when not specified.
Bug: 206718630
Bug: 206718630
Test: aidl --lang cpp -o out -h out \
tests/android/aidl/tests/StructuredParcelable.aidl
Test: aidl --lang ndk -o out -h out \
tests/android/aidl/tests/StructuredParcelable.aidl
Merged-In: I15cfe2a90c0c9adca1692d9a9c997901b90bbbb6
Change-Id: I15cfe2a90c0c9adca1692d9a9c997901b90bbbb6
(cherry picked from commit 94d3c0c503ad14f23ad1dce4bfd8020e031657a4)
Merged-In: I15cfe2a90c0c9adca1692d9a9c997901b90bbbb6
|
|
Change-Id: I84147691e9b2282791640ba3b2f29d5f3f393e73
|
|
This change fixes a bug that when -o option is used without explicitly
specifying the output path, the output path is determined by
concatenating the directory specified by the -o option and the input
file path and the replacing the suffix .aidl to .java.
This especially is problematic in Windows, because the output path then
contains drive letter (e.g. "C:") in the middle of the path if the input
path is an absolute path. For example,
aidl -o C:\output C:\input\IFoo.aidl
resulted the incorrect output path of C:\output\C:\input\IFoo.java.
It should be C:\input\IFoo.java
This change fixes the problem by not concatenating the output dir with
the input file path.
Bug: 134966834
Test: aidl_unittests
Merged-In: I417623089f7409032483a2f6c79e745c0fc340d3
(cherry picked from commit 56f73d76e064a0bba1a103e5c37bc3d5d57860f9)
Change-Id: I417623089f7409032483a2f6c79e745c0fc340d3
|
|
Change-Id: Id471f7d6f9baaacf96d57b39e834cf5cd6457342
|
|
The API signature file now contains warning preamble as comments. The
comments shouldn't fail 'diff'.
Bug: 134452637
Test: m out/soong/.intermediates/frameworks/base/services/net/ipmemorystore-aidl-interfaces-api/checkapi_dump.timestamp
with ag/7952481 applied in qt-dev.
Merged-In: I5547b768c216ee67e7d2d3b82752a94fea27004c
(cherry picked from commit 8558309645adf15978321528b71a60e45731a250)
Change-Id: I5547b768c216ee67e7d2d3b82752a94fea27004c
|
|
We have found that people tend to ignore build errors and force update
the frozen AIDL API signature files. To better prevent it from
happening, this change
- Makes the error message more visible and explicit. The message also
explains why force updating the API signature files are not allowed.
- Put similar warning messages to the API signature files as comment
when the new signature API files are created.
Bug: 134452637
Test: add a random method to IBaz.aidl and run `m test-root-package`
"ERROR: AIDL API change detected on a released platform" is shown.
Test: `m test-root-package-freeze-api`
build/api/3/test_package/IBaz.aidl contains the warning message
(cherry picked from commit f1bc25ed27703e69ff87ecc1c59e204eac2297cf)
Merged-In: I860620daa61770a68990456fafd9e5950fde9405
Change-Id: I860620daa61770a68990456fafd9e5950fde9405
|
|
Change-Id: Id2fff82bfa216625303dc9d733c2cfc0a61de0a5
|
|
Java:
- Server
writeNoException(), and then writeInt(version)
- Client
As-is: just readInt() -> read exception code, not version
To-be: readException, and then readInt() -> consume exception code first
Cpp:
- Transaction code
As-is: getInterface's ID
To-be: FIRST_CALL_TRANSACTION + the ID
- Server & Client
As-is: No exception read/write before version code
To-be: Add exception value before version code
Bug: 133118233
Test: m
Test: atest CtsNdkBinderTestCases
Test: ./runtests.sh
Change-Id: I168a0b8ff7e1871afd95a0ab76b544bfe131a9ed
Merged-In: I168a0b8ff7e1871afd95a0ab76b544bfe131a9ed
|
|
Change-Id: Ic4048f739b26e34aa6d55444e239d95e4e5ea5dd
|
|
* changes:
AIDL NDK logger bugfix
Log supporting for binder ndk
|
|
1. When the error occurs in server side, it exits before log, so make
logger called before exit.
2. service_specific_code had wrong data, so change it to valid code.
Bug: 126501406
Test: m
Change-Id: Ibbb072a067ab2c16f3fdefcc777c4900fb8492ae
Merged-In: Ibbb072a067ab2c16f3fdefcc777c4900fb8492ae
|
|
Move log code Generating code aidl_to_cpp -> aidl_to_common
Add dependency of libjsoncpp(_ndk)
-ndk: libjsoncpp_ndk
-ndk_platform: libjsoncpp
Bug: 126501406
Test: m
Test: ./runtests.sh
Change-Id: Ife69f146ce313564082d070e5e047ec8e1111093
Merged-In: Ife69f146ce313564082d070e5e047ec8e1111093
|
|
Change-Id: Ibfc86f5532ce67eac97a0aa51199a87d46d1ca0c
|
|
APIs must match ToT so that we know they don't change and we can rely on
them. However, this is for a test, so it doesn't matter in practice.
This moves forward the release process, but we should look at avoiding
needing test APIs frozen.
Bug: 131924172
Test: force checkEquality to be called in aidl_interface.go, repro build
error reported by release team, update APIs to fix error
Change-Id: Iabec6e853655b5f535b35f6e16b53f782f609b24
|
|
Change-Id: I8608bc6fcdcd58863e817fc2d864ac852f8ba077
|
|
finalized" into qt-dev
|
|
ParcelFileDescriptor is treated as a built-in type, but it's also in the framework.aidl.
So aidl ignores its declaration in framework.aidl to prevent duplication
Bug: 130899491
Test: m
Test: check if "attempt to redefine built in class android.os.ParcelFileDescriptor" doesn't show
Change-Id: Iffcd6f6a5348abb59a2df481883ca48493f2ed66
Merged-In: Iae33b1543f81fdc68ee20588f705f03204c14502
|
|
When the SDK is finalized, modification (including backward compatible
changes) to the AIDL interfaces is prohibited.
Modification can only be made in the non-finalized branch, e.g. master,
by introducing a new version.
Bug: 130011602
Test: manual
Set PLATFORM_VERSION_CODENAME to REL and `m netd_aidl_interface-cpp`
Fails with following error.
Check equality of out/soong/.intermediates/system/netd/server/netd_aidl_interface-api/dump and system/netd/server/binder/api/1
FAILED: out/soong/.intermediates/system/netd/server/netd_aidl_interface-api/checkapi_dump.timestamp
diff -r system/netd/server/binder/api/1 out/soong/.intermediates/system/netd/server/netd_aidl_interface-api/dump && touch out/soong/.intermediates/system/netd/server/netd_aidl_interface-api/checkapi_dump.timestamp
diff -r system/netd/server/binder/api/1/android/net/INetd.aidl out/soong/.intermediates/system/netd/server/netd_aidl_interface-api/dump/android/net/INetd.aidl
92a93,96
> void firewallAddUidInterfaceRules(in @utf8InCpp String ifName, in int[] uids);
> void firewallRemoveUidInterfaceRules(in int[] uids);
> void trafficSwapActiveStatsMap();
Merged-In: I6229e6e6d19153564939e9fb9ea19a1177c2279c
Change-Id: I6229e6e6d19153564939e9fb9ea19a1177c2279c
(cherry picked from commit 7c3c11c36a6b0b39c565b009a50fdfb0f7fe6889)
|
|
Change-Id: I39af33f3b957951013b00555feb7f0bfbe7a4c44
|
|
|
|
|
|
The two methods are now with @hide as they are not intended for public
use.
Bug: 124210145
Test: m
Test: aidl_unittests
Change-Id: I8665f11e8912159d37f0d54e643df25cdacf9e16
|
|
Fix a bug that a meta transaction incorrectly turns hasUnassignedIds on.
Also add a few more unit tests for the ID assignments.
Bug: 130328570
Test: aidl_unittests
Merged-In: I375ec8fdf9f633bc71f7d5d03f564789f563b30e
Change-Id: I375ec8fdf9f633bc71f7d5d03f564789f563b30e
(cherry picked from commit a1bdb7364a2d2e9c417686b4b84cba443e2d1bca)
|
|
Change-Id: I4a6cce6a425f5898538394af0b2c6fa70cb3485d
|
|
This change fixes a bug that getInterfaceVerison() is not implemented in
the stub class when --lang=cpp and --version is set.
Bug: 130195984
Test: m
Test: aidl_unittests
Merged-In: I2b107db8c47ef287ec2be736b6beb84768cb04e6
Change-Id: I2b107db8c47ef287ec2be736b6beb84768cb04e6
(cherry picked from commit 22d4cfc8fbef14080ce082e9e387ed6da2fa1321)
|
|
Change-Id: Iee0a1d1399e507c9f0107148eafc9ab11377d258
|
|
|
|
As-is: Object type {a:1, b:2, c:3}
To-be: Array type [{name: a, value:1}, {name:b, value:2}, {name:c, value:3}]
Object type doesn't keep insertion order. So change it to array type
Bug: 128833454
Test: m -j
Change-Id: Ic7c36560de03a4f2bbefef3c4a0a790ce4b90869
Merged-In: Ic7c36560de03a4f2bbefef3c4a0a790ce4b90869
(cherry picked from commit 2a2581ea3a58c92a429db85485e74c692dd99de4)
|
|
Previously, the generated source files of an aidl_interface were found
under:
out/soong/.intermediate/<path_to_module>/<module>-cpp-gen-<number>/gen
, where <number> is assigned arbitrarily. This caused a problem when the
paths are added to the source path of an IDE. The paths had to be
updated whenever the assigned numbers are changed.
Now, the paths get simpler:
out/soong/.intermediate/<path_to_module>/<module>-cpp-source/gen
This is done by building multiple source files using a single genrule
module.
Bug: 130011113
Test: m
Test: system/tools/aidl/runtests.sh
Merged-In: I308f3c3ced5b2f7ca3a95ea3fd8ab49d8dd9105b
Change-Id: I308f3c3ced5b2f7ca3a95ea3fd8ab49d8dd9105b
(cherry picked from commit b8dceed62c9383b50b4a1f2525c49d78d152b3cf)
|
|
Change-Id: I938d7ad0dbfea95c02ed7e67a448ae0fb2d2a552
|
|
This reverts commit cc14a584a301533653639d9f6847111ca8a79c68.
Bug: 129549755
Test: m
Test: m libvirtualtouchpadclient
Test: ./runtests.sh
Merged-In: Id0103c1f05654c87a3eac449c985c805ae963d0e
Change-Id: Id0103c1f05654c87a3eac449c985c805ae963d0e
(cherry picked from commit 5b7e532899a56e88e5fbf53ffd6686d941be7da9)
|
|
Change-Id: I31bd84db4f3674a0efeecdddcbe04f60d57062a9
|
|
2133f7fdd0
am: 4aa7867b72
Change-Id: I969ee261037d1baa9e2e9227b10bc428b9b200ad
|
|
am: 2133f7fdd0
Change-Id: I3638bfa98080d8f66b0726c06efde9b511e2c301
|
|
am: cc14a584a3
Change-Id: I7d023dd58facc31a55154086ad1036fd64568f30
|
|
This reverts commit 7df2c34b6e40828fdf362c665911173e188fe1c4.
Reason for revert: Possible cause of break in stage-aosp-master
Change-Id: I89f2c1c5b645c2434eaac149c32b8220555225e4
|
|
am: 1b0ba5eb7a
Change-Id: Ic51874427d4e0603179db35d273cd47d22412bcc
|
|
am: acc4190f0a
Change-Id: I9e928e1bcb89606d35f05227f9f904f9ef670bb4
|
|
am: 7df2c34b6e
Change-Id: Iaec077af6a9e114a7e6881f2294e279b31e220ab
|
|
This is to be in sync with the rule in aidl_interface.go.
Bug: 129549755
Test: m
Change-Id: I5bd5bec380991dd6340d569706198242f7ad3d79
|
|
Change-Id: Ic66df508c6a6a7720954cd2eece4e9d679ced128
|
|
am: abc414c0dd
Change-Id: Id9dbf76ec06fb095b90f1745b2f420ca6d0b166c
|
|
am: ef9d233b48
Change-Id: Ifdb51337efe2535486bc3cf6a42374dd5696caec
|
|
am: de138911c8
Change-Id: I9167c72fd1bc646a1d266dd32b6470fe101074e9
|
|
Change-Id: I54debe9bd02ac69ed6899a98e14cf2f3704d2385
|
|
Generated by clang-tidy.
Test: m checkbuild
Change-Id: I9edd67d241453ebca9a3e84c72e2b122e72cd5df
|
|
am: 8123a1d001
Change-Id: I8499967f157de347b5b8696554d65f06353ac2c8
|
|
am: 09e3c582ed
Change-Id: I35efbcba5298acd2d850584bfea2ec5600866bad
|
|
am: 6e4353264f
Change-Id: I85b04ccb763f0bbc776ee3e5293102984da00854
|