Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit b69df5e1 authored by Omair Kamil's avatar Omair Kamil
Browse files

Remove unused loop parameter in ScanManager and add comment on its intended behavior. 

Bug: 327671856
Flag: EXEMPT - no logic change
Test: atest BluetoothInstrumentationTests
Change-Id: I1a0f8aa6078e7a9f58f67626792718906cebd272
parent c67dc445

android/app/src/com/android/bluetooth/le_scan/ScanManager.java

+8 −5
Original line number Diff line number Diff line
@@ -1354,11 +1354,14 @@ public class ScanManager { 
            }

            int deliveryMode = getDeliveryMode(client);

            if (deliveryMode == DELIVERY_MODE_ON_FOUND_LOST) {

                for (ScanFilter filter : client.filters) {

                    int entriesToFree = getNumOfTrackingAdvertisements(client.settings);

                    if (!manageAllocationOfTrackingAdvertisement(entriesToFree, false)) {

                        Log.e(TAG, "Error freeing for onfound/onlost filter resources "

                                + entriesToFree);

                // Decrement the count of trackable advertisements in use

                int entriesToFreePerFilter = getNumOfTrackingAdvertisements(client.settings);

                for (int i = 0; i < client.filters.size(); i++) {

                    if (!manageAllocationOfTrackingAdvertisement(entriesToFreePerFilter, false)) {

                        Log.e(

                                TAG,

                                "Error freeing for onfound/onlost filter resources "

                                        + entriesToFreePerFilter);

                        try {

                            mScanHelper.onScanManagerErrorCallback(

                                    client.scannerId, ScanCallback.SCAN_FAILED_INTERNAL_ERROR);