Warnings when compiling this code

[roleroz]

When you build the code in this library there are several warnings printed

• Order member initialization on BLELocalCharacteristic (fixed by Fix warnings #302)
• Unused variables in: (fixed by Fix warnings #302)
  • HCIClass::tryResolveAddress
  • HCIClass::handleEventPkt (the whole else if statement did nothing)
  • L2CAPSignalingClass::handleSecurityData (one initialization should be inside of BLE_TRACE)
  • BluetoothCryptoToolbox::f5
  • BluetoothCryptoToolbox::testAh
  • BluetoothCryptoToolbox::testg2
• Uninitialized variable in L2CAPSignalingClass::handleSecurityData (fixed by Fix warnings #302)
• Unenumerated case in ArduinoBLE/src/utility/HCI.cpp (would be fixed by merge of fix(HCI): missing LE_Event_Mask #386)

I have a PR ready for fixing most of these warnings (all except the last one) in https://github.com//pull/302, but I haven't had luck getting any traction on getting that reviewed

If someone could review that PR most of these things would be fixed.

If someone tells me what 0x0A means in that unenumerated case, I'm happy to add that to the enum as well

Additional context

Additional reports

• Lots of Compiler Warnings - At least if I build for a Teensy 4.1 or Micromod #274
• HCI.cpp:991:7: error: case value '10' not in enumerated type 'LE_META_EVENT' #346

Related

• warning: variable length array 'txBuffer' is used [-Wvla] #89 / Allocate dyamic sized arrays #141
• Compiler warning: "'us_timestamp_t mbed::TimerBase::read_high_resolution_us() const' is deprecated" #164

[roleroz]

This is still incomplete, there is also the unenumerated case in ArduinoBLE/src/utility/HCI.cpp, if someone can tell me what does 0x0A means in that file, I'll create a PR for it

[ataweg]

Issue "Unenumerated case in ArduinoBLE/src/utility/HCI.cpp" not fixed . Depending on the Preferences "Compiler warnings" set to All or More there is an error message:

f:\Projects\Arduino\libraries\ArduinoBLE\src\utility\HCI.cpp: In member function 'virtual void HCIClass::handleEventPkt(uint8_t, uint8_t*)':
f:\Projects\Arduino\libraries\ArduinoBLE\src\utility\HCI.cpp:991:7: error: case value '10' not in enumerated type 'LE_META_EVENT' [-Werror=switch]
       case 0x0A:{
       ^~~~
cc1plus.exe: some warnings being treated as errors

[fpistm]

This 0x0A value was introduced with PR #156
It seems to be a the same than CONN_COMPLETE with only 2 fields added in the struct:

          uint8_t localResolvablePrivateAddress[6];
          uint8_t peerResolvablePrivateAddress[6];

But never filled. Only print when BLE_TRACE is enabled.

Maybe @unknownconstant could comment on this?

[unknownconstant]

Let me take a look - it's been a while since I've been through this code