Age | Commit message (Collapse) | Author |
|
I did my shopping list of errorprone that only appear as warning and are
note enforced yet. I am promoting them to error to make sure this will
never happen
Bug: 344658662
Test: m .
Flag: Exempt refactor
Change-Id: I7a4e73c92a094d14fc12c1a801d6fb5fb386ae6c
|
|
Bug: 344658662
Test: m BluetoothInstrumentationTests
Test: m .
Flag: Exempt refactor
Change-Id: Iea773640f276794f32020685feb886b53485a0c1
|
|
Bug: 344658662
Test: m BluetoothInstrumentationTests
Test: m .
Flag: Exempt refactor
Change-Id: I0870dce29ea4f2db598e0b7ed12d59da0b7a7e68
|
|
|
|
|
|
Bug: 394958283
Test: atest AdvtFilterOnFoundOnLostInfoTest
Flag: Exempt refactor no-op
Change-Id: If4453a18e0241f9609dad5a7588c67ed9ef4bec9
|
|
Also fix suppress of TruthIncompatibleType that were related
Bug: 311772251
Test: m BluetoothInstrumentationTests
Flag: TEST_ONLY
Change-Id: Ic63c26bcf9411e1670684b17f05b4fe1785b9a10
|
|
Bug: 344658662
Test: m BluetoothInstrumentationTests
Test: m .
Flag: Exempt refactor
Change-Id: I8c487c01d3b67a6a833960a337e5a507bdd89afa
|
|
Bug: 344658662
Test: m BluetoothInstrumentationTests
Test: m .
Flag: Exempt refactor
Change-Id: Ibde1dabbb1ba64cd278e9607e29dec04e70b1bb9
|
|
This reverts commit fe8ca2cd5852d9ab88bde510b4803288a96449e2.
Reason for revert: Breaking git_main-throttled-monitored on test_suites_riscv-trunk_staging. The bug is non-deterministic so it was assumed to be fixed in local testing.
Bug: 398000890
Change-Id: Ie21209eea4f191aa20576872bd4b25ae40614b2f
|
|
This reverts commit 7a59be26ec685dec6f4853f5cb4cc27e73a13ba3.
The new clang update (r547379) does not have the linker relaxation bug
anymore so we can revert this workaround.
Bug: 390197215
Test: m net_test_stack and mmma packages/modules/Bluetooth
Change-Id: I10260faaf2d249afffe0ff406bc27c2931817794
|
|
|
|
Bug: 374984156
Test: m com.android.bt
Flag: EXEMPT, build change
Change-Id: Ibc8bad33a3e5712a9a19040fcbaa36e1f0d02d9e
|
|
|
|
Bug: 311772251
Test: m BluetoothInstrumentationTests
Flag: TEST_ONLY
Change-Id: I9b6b3a59133520d485909cd4dedb6e6f189863d5
|
|
Bug: 393400512
Change-Id: I7fb9ecbaf1510fc5f1b8eb184554db10a1ebc5b5
Test: None
Flag: TEST_ONLY
|
|
|
|
Bug: b/386842159
Test: m checkbuild
Change-Id: I3cadd1973c60077efd5619ed65e9ec7748c6f1fc
|
|
Bug: 311772251
Test: m .
Flag: Exempt no-op change to respect lint recommendation
Change-Id: I06880640026315e1aa20f3d0aadddd6e50804666
|
|
Bug: 311772251
Test: m .
Flag: Exempt no-op change to respect lint recommendation
Change-Id: Ida99e0e524be3edac9ae78cbb0b26aa98e693282
|
|
Also fixed the reported warning
Bug: 311772251
Test: m .
Flag: Exempt refactor
Change-Id: I99eb7e12468d2b07982f7ce69a53cc0c5e2c9d12
|
|
Bug: 360924438
Test: mma -j32
Flag: com.android.bluetooth.flags.hci_vendor_specific_extension
Change-Id: I02baab09782a0e8fc88d36c7d9bc4c4fa518fb8c
|
|
|
|
One is going down to the native, the other from. This allow to have a
couple final method in the nativeCallback and remove the need for null
pointer check (by design)
Similar as HAP implementation
https://r.android.com/2953314 -> direct use of native interface
https://r.android.com/3199943 -> callback split
Bug: 311772251
Bug: 361263965
Test: atest BluetoothInstrumentationTests
Test: atest VolumeControlNativeCallbackTest
Flag: com.android.bluetooth.flags.leaudio_add_aics_support
Change-Id: Ia167c8db9f5de5f9dd6508abde1bf9b8e4b0cbb8
|
|
Bug: 374984156
Test: m com.android.btservices
Test: WITH_TIDY_FLAGS=--fix m libbtif
Flag: EXEMPT, no logical change
Change-Id: I1f89824743dd57ca6961c513e460603623d1f117
|
|
Bug: 344658662
Test: m com.android.btservice
Flag: Exempt refactor annotation only
Change-Id: I20d951cb49b289b6eb880cb8d175f62242b1306c
|
|
Bug: 344658662
Test: m com.android.btservices
Flag: Exempt lint enforcement
Change-Id: I2951e2a779749f181ff713f923c098d838ba0965
|
|
|
|
Bug: 344658662
Test: m com.android.btservices
Flag: Exempt lint enforcement
Change-Id: I3674436976123e6c4b7e774f06500372baa3eecb
|
|
Enforces clang-tidy checks for all cc targets
Bug: 331817295
Test: m com.android.btservices
Flag: EXEMPT, build change
Change-Id: I91f0a12aa06d12ad1dcbdf23ae67d3948ac66c25
|
|
This lint check will raise an error if a function is implemented
without a corresponding prototype declaration included in scope.
The main use case will be to ensure mock functions do not become out of
sync causing the original function to be linked instead.
Because of the large amount of errors, this change suppresses
errors in all locations, to be cleaned up gradually.
Bug: 369381361
Test: m com.android.btservices
Flag: EXEMPT, lint fixes
Change-Id: I888b5e284e237d8a6e2864bdde55c0a7fc0c348d
|
|
Bug: 344658662
Test: m .
Change-Id: I50f6d2a3b677f965ae0d977528c6d060db8b90ff
|
|
Test: mmm packages/modules/Bluetooth
Flag: EXEMPT, build change
Bug: 315250603
Change-Id: Ia4b367609f278103494c3870b17bea6f127b1f7c
|
|
Bug: 305066880
Test: mma -j32
Flag: EXEMPT, build infrastructure change
Change-Id: Ic60b8a4959e013fd2455a4c85659d4d0f7052b2e
|
|
The underlying issue seems to have been fixed
Bug: 154347133
Fix: 308547175
Test: N/A
Change-Id: I8ca8a8c5328c876b6410f8a4c85f614d5feafb89
|
|
Since we fixed all warning, it would be better to not introduce any
futur regression, hence adding the flag by default
Bug: 297606425
Test: m Bluetooth | this is no-op, only impact build
Change-Id: Iac0b15bae84408804c5f9c1d4428a3201484d500
|
|
Bug: 299772495
Test: mmm packages/modules/Bluetooth
Change-Id: I71342aafd09d4e2799e320d9e68efdb51105f52c
|
|
Test: mmm packages/modules/Bluetooth
Bug: 279502784
Change-Id: Ic5bf6a1128a0fdf9bb20616189b9f53d00ea5071
|
|
fluoride_common_options are not only used inside fluoride but all
bluetooth cc targets so use the prefix `bluetooth` instead of
`fluoride`.
Also use a less generic name `common_options` -> `cflags` to
document what contains this default for the rules using this default
Test: mma packages/modules/Bluetooth
Bug: 279502784
Change-Id: Ic3b7a62bc97f8eab24d9130fbe2e2367596bdc9b
|
|
|
|
95ced0c74f790d8083dfce78fa034d83c0aa7b03
Change-Id: Ie9b3c8511fb8c18203cc98f4bee8c427824bd897
|
|
Revert submission 2510997-revert-2508278-NYSMMYRTQW
Reason for revert: Fixed with https://r.android.com/2509792
Reverted changes: /q/submissionid:2510997-revert-2508278-NYSMMYRTQW
Change-Id: I28a82d185b15ba1f1364fc9c40fda86d9f506cd7
|
|
Revert submission 2508278
Reason for revert: Breaks many builds
Reverted changes: /q/submissionid:2508278
Bug: 275453018
Change-Id: I496d01595c06921933a3cd906d5b840b7b94f4c6
|
|
Revert submission 2508278
Reason for revert: Breaks many builds
Reverted changes: /q/submissionid:2508278
Bug: 275453018
Change-Id: I56ecc1cae61b19f052b707beab55ce4590eaa8f8
|
|
Address Sanitizer seems to only be flaky on 64bit binaries.
So workaround this issue by running the 32bit binaries on 64bit targets.
Bug: 273846603
Bug: 154347133
Test: TreeHugger
Change-Id: Ic573b01c122d44eb6928516682305bfec53fb957
|
|
This reverts commit e58cc9d43c2878bb56b73b4c62ee944b502efd34.
Reason for revert: Trying another workarround, reverting to reduce
the number of configuration possibilities
Test: None
Bug: 273588072
Bug: 273846603
Change-Id: I9b2eee0f460b59f6d806b33ceebe2f1fda4d3e43
|
|
Test: m
Change-Id: Idae4a53de77ef2757dba38dec931900a988c1b6a
|
|
Those modules do not contain a lot of tests, so sharding
them spread errors to multiple shards.
Tag: Stability
Test: presubmit
Bug: 273588072
Bug: 273846603
Change-Id: I819dbc33d2c23f590315bd8cbaebfd0ac0739f7a
|
|
Bug: 215008564
Test: manual
Tag: #feature
Ignore-AOSP-First: Some changes only present in internal
Change-Id: I0c12526bdf833b41e8d1282b29640fa9481b6a04
|
|
Added SPDX-license-identifier-Apache-2.0 to:
apex/Android.bp
audio_a2dp_hw/Android.bp
audio_bluetooth_hw/Android.bp
audio_hal_interface/Android.bp
audio_hearing_aid_hw/Android.bp
binder/Android.bp
bta/Android.bp
btcore/Android.bp
btif/Android.bp
build/Android.bp
common/Android.bp
conf/Android.bp
device/Android.bp
embdrv/g722/fuzzer/Android.bp
embdrv/sbc/Android.bp
embdrv/sbc/decoder/Android.bp
embdrv/sbc/encoder/Android.bp
gd/att/Android.bp
gd/common/Android.bp
gd/dumpsys/Android.bp
gd/dumpsys/bundler/Android.bp
gd/fuzz/Android.bp
gd/hal/Android.bp
gd/hci/Android.bp
gd/iso/Android.bp
gd/l2cap/Android.bp
gd/l2cap/fuzz/Android.bp
gd/neighbor/Android.bp
gd/os/Android.bp
gd/packet/Android.bp
gd/packet/parser/Android.bp
gd/packet/parser/test/Android.bp
gd/proto/Android.bp
gd/rust/common/Android.bp
gd/rust/facade/Android.bp
gd/rust/gddi/Android.bp
gd/rust/hal/Android.bp
gd/rust/hci/Android.bp
gd/rust/link/Android.bp
gd/rust/main/Android.bp
gd/rust/shim/Android.bp
gd/security/Android.bp
gd/security/channel/Android.bp
gd/security/pairing/Android.bp
gd/security/record/Android.bp
gd/shim/Android.bp
gd/storage/Android.bp
hci/Android.bp
include/Android.bp
internal_include/Android.bp
main/Android.bp
main/shim/Android.bp
osi/test/fuzzers/Android.bp
osi/test/fuzzers/alarm/Android.bp
osi/test/fuzzers/allocation_tracker/Android.bp
osi/test/fuzzers/allocator/Android.bp
osi/test/fuzzers/array/Android.bp
osi/test/fuzzers/buffer/Android.bp
osi/test/fuzzers/compat/Android.bp
osi/test/fuzzers/fixed_queue/Android.bp
osi/test/fuzzers/future/Android.bp
osi/test/fuzzers/list/Android.bp
osi/test/fuzzers/ringbuffer/Android.bp
packet/Android.bp
packet/avrcp/Android.bp
packet/base/Android.bp
packet/tests/fuzzers/Android.bp
profile/avrcp/Android.bp
profile/sdp/Android.bp
service/Android.bp
service/common/Android.bp
stack/test/fuzzers/Android.bp
stack/test/fuzzers/a2dp/Android.bp
stack/test/fuzzers/a2dp/codec/Android.bp
stack/test/fuzzers/sdp/Android.bp
test/Android.bp
test/headless/Android.bp
test/rootcanal/Android.bp
test/suite/Android.bp
types/Android.bp
udrv/Android.bp
utils/Android.bp
vendor_libs/Android.bp
vendor_libs/linux/Android.bp
vendor_libs/linux/interface/Android.bp
vendor_libs/test_vendor_lib/Android.bp
vendor_libs/test_vendor_lib/types/Android.bp
Added SPDX-license-identifier-Apache-2.0 SPDX-license-identifier-BSD to:
gd/Android.bp
gd/crypto_toolbox/Android.bp
stack/Android.bp
Added SPDX-license-identifier-Apache-2.0 SPDX-license-identifier-BSD
SPDX-license-identifier-MIT legacy_unencumbered
to:
Android.bp
Added SPDX-license-identifier-Apache-2.0 SPDX-license-identifier-BSD
legacy_unencumbered
to:
embdrv/g722/Android.bp
Added SPDX-license-identifier-Apache-2.0 SPDX-license-identifier-MIT
to:
osi/Android.bp
Bug: 68860345
Bug: 151177513
Bug: 151953481
Test: m all
Exempt-From-Owner-Approval: janitorial work
Change-Id: I883bd6b68223a5bc77fdac89e0280993f617d6d0
|