From a48c6d4f40312720c074ec8aa2722c178d25d42b Mon Sep 17 00:00:00 2001 From: paul-szczepanek-arm <33840200+paul-szczepanek-arm@users.noreply.github.com> Date: Tue, 27 Nov 2018 15:38:01 +0000 Subject: [PATCH] consistent errors --- features/FEATURE_BLE/source/generic/GenericGap.cpp | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/features/FEATURE_BLE/source/generic/GenericGap.cpp b/features/FEATURE_BLE/source/generic/GenericGap.cpp index 70e2390239..49d484b065 100644 --- a/features/FEATURE_BLE/source/generic/GenericGap.cpp +++ b/features/FEATURE_BLE/source/generic/GenericGap.cpp @@ -2053,7 +2053,7 @@ ble_error_t GenericGap::destroyAdvertisingSet(advertising_handle_t handle) return BLE_ERROR_INVALID_PARAM; } - if (_existing_sets.get(handle) == false) { + if (!_existing_sets.get(handle)) { return BLE_ERROR_INVALID_PARAM; } @@ -2362,7 +2362,7 @@ ble_error_t GenericGap::stopAdvertising(advertising_handle_t handle) } if (!_active_sets.get(handle)) { - return BLE_ERROR_INVALID_PARAM; + return BLE_ERROR_INVALID_STATE; } ble_error_t status; @@ -2431,7 +2431,7 @@ ble_error_t GenericGap::setPeriodicAdvertisingParameters( } if (!_existing_sets.get(handle)) { - return BLE_ERROR_INVALID_STATE; + return BLE_ERROR_INVALID_PARAM; } return _pal_gap.set_periodic_advertising_parameters( @@ -2458,7 +2458,7 @@ ble_error_t GenericGap::setPeriodicAdvertisingPayload( } if (!_existing_sets.get(handle)) { - return BLE_ERROR_INVALID_STATE; + return BLE_ERROR_INVALID_PARAM; } if (payload.size() > getMaxAdvertisingDataLength()) { @@ -2515,7 +2515,7 @@ ble_error_t GenericGap::startPeriodicAdvertising(advertising_handle_t handle) } if (!_existing_sets.get(handle)) { - return BLE_ERROR_INVALID_STATE; + return BLE_ERROR_INVALID_PARAM; } if (_active_sets.get(handle) == false) { @@ -2548,7 +2548,7 @@ ble_error_t GenericGap::stopPeriodicAdvertising(advertising_handle_t handle) } if (!_existing_sets.get(handle)) { - return BLE_ERROR_INVALID_STATE; + return BLE_ERROR_INVALID_PARAM; } if (_active_periodic_sets.get(handle) == false) {