FIX #197 : Replace varclr with arrclr for BLEAdvertising _data fields

CSC-Sendance · CSC-Sendance · commit dc1b3d75154d · 2023-07-24T14:07:51.000+02:00
Otherwise the data field gets corrupted (?) causing error msg "bool BLEAdvertising::_start(uint16_t, uint16_t): 364: verify failed, error = NRF_ERROR_INVALID_LENGTH" as mentioned in this comment: #197 (comment) see also https://devzone.nordicsemi.com/f/nordic-q-a/52573/sd_ble_gap_adv_set_configure-returns-error-nrf_error_invalid_length
diff --git a/libraries/Bluefruit52Lib/src/BLEAdvertising.cpp b/libraries/Bluefruit52Lib/src/BLEAdvertising.cpp
@@ -42,7 +42,7 @@
 BLEAdvertisingData::BLEAdvertisingData(void)
 {
   _count = 0;
-  varclr(_data);
+  arrclr(_data);
 }
 
 bool BLEAdvertisingData::addData(uint8_t type, const void* data, uint8_t len)
@@ -242,7 +242,7 @@ bool BLEAdvertisingData::setData(uint8_t const * data, uint8_t count)
 void BLEAdvertisingData::clearData(void)
 {
   _count = 0;
-  varclr(_data);
+  arrclr(_data);
 }
 
 /*------------------------------------------------------------------*/

Original file line number	Diff line number	Diff line change
`@@ -42,7 +42,7 @@`
`42`	`42`	`BLEAdvertisingData::BLEAdvertisingData(void)`
`43`	`43`	`{`
`44`	`44`	`_count = 0;`
`45`		`- varclr(_data);`
	`45`	`+ arrclr(_data);`
`46`	`46`	`}`
`47`	`47`
`48`	`48`	`bool BLEAdvertisingData::addData(uint8_t type, const void* data, uint8_t len)`
`@@ -242,7 +242,7 @@ bool BLEAdvertisingData::setData(uint8_t const * data, uint8_t count)`
`242`	`242`	`void BLEAdvertisingData::clearData(void)`
`243`	`243`	`{`
`244`	`244`	`_count = 0;`
`245`		`- varclr(_data);`
	`245`	`+ arrclr(_data);`
`246`	`246`	`}`
`247`	`247`
`248`	`248`	`/------------------------------------------------------------------/`