BLE: correctness improvments

Remove unused variable, improve header inclusion, fix initialization list order,
pull/13475/head
Vincent Coubard 2020-09-04 13:26:29 +01:00
parent 702c1b9756
commit f116604024
3 changed files with 3 additions and 7 deletions

View File

@ -68,7 +68,7 @@ public:
/**
* Construct a AdvertisingDataSimpleBuilder
*/
AdvertisingDataSimpleBuilder() : _builder(_buffer), _buffer()
AdvertisingDataSimpleBuilder() : _buffer(), _builder(_buffer)
{
}

View File

@ -991,8 +991,6 @@ uint8_t GattServer::atts_write_cb(
attsAttr_t *pAttr
)
{
uint8_t err;
GattCharacteristic* auth_char = getInstance().get_auth_char(handle);
if (auth_char && auth_char->isWriteAuthorizationEnabled()) {
GattWriteAuthCallbackParams write_auth_params = {

View File

@ -37,11 +37,7 @@
#include "ble/gap/Events.h"
#include "source/pal/PalGap.h"
#include "source/pal/GapEvents.h"
#include "source/pal/GapTypes.h"
#include "source/pal/PalEventQueue.h"
#include "source/pal/PalConnectionMonitor.h"
#include "source/pal/PalEventQueue.h"
#include "ble/Gap.h"
@ -53,6 +49,8 @@ class PalSecurityManager;
class PalGap;
class PalEventQueue;
namespace impl {
class BLEInstanceBase;