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

Commit c9db2d34 authored by Jakub Pawlowski's avatar Jakub Pawlowski
Browse files

Remove unnecessary advertiser count check

Bluetooth stack won't allow to register more advertisers that it can
handle, so checking count of registered advertisers make no sense when
calling startAdvertising.

Bug: 30622771
Test: sl4a ConcurrentBleAdvertisingTest
Change-Id: Ifbaf583871cf59ce8e345d7bca8e9a9ea1540734
parent 54b819dc
Loading
Loading
Loading
Loading
+0 −10
Original line number Diff line number Diff line
@@ -201,11 +201,6 @@ class AdvertiseManager {
                return;
            }

            if (mAdvertiseClients.size() >= maxAdvertiseInstances()) {
                postCallback(advertiserId,
                        AdvertiseCallback.ADVERTISE_FAILED_TOO_MANY_ADVERTISERS);
                return;
            }
            if (!mAdvertiseNative.startAdverising(client)) {
                postCallback(advertiserId, AdvertiseCallback.ADVERTISE_FAILED_INTERNAL_ERROR);
                return;
@@ -230,11 +225,6 @@ class AdvertiseManager {
                mAdvertiseClients.remove(client);
            }
        }

        // Returns maximum advertise instances supported by controller.
        int maxAdvertiseInstances() {
            return mAdapterService.getNumOfAdvertisementInstancesSupported();
        }
    }

    // Class that wraps advertise native related constants, methods etc.