BLE C++ Wrapper

RE: BLE C++ Wrapper

Sebastien Huet — Mon, 11 Sep 2017 07:13:52 GMT

in your header:

...
public:
    static BleWrapper &getInstance() {
   static BleWrapper instance{};
   return instance;
}
    void init(void);
    void start(void);
...

RE: BLE C++ Wrapper

schef — Mon, 11 Sep 2017 07:09:56 GMT

How would i implement this instead of my code?

RE: BLE C++ Wrapper

Sebastien Huet — Mon, 11 Sep 2017 06:53:05 GMT

this is thread safe when using c++11 and up. your are sure to have only one instance.

RE: BLE C++ Wrapper

schef — Mon, 11 Sep 2017 06:46:34 GMT

Thanks for the suggestion. What do you mean by stronger?

RE: BLE C++ Wrapper

Sebastien Huet — Mon, 11 Sep 2017 06:35:11 GMT

Nice work, thanks.

you could construct your singleton like below, this is stronger.

static BleWrapper &getInstance() {
   static BleWrapper instance{};
   return instance;
}

RE: BLE C++ Wrapper

schef — Mon, 11 Sep 2017 06:08:41 GMT

I moved to sdk14 but here you go. gist.github.com/.../f945611d3835dc708371269407b29c6b

RE: BLE C++ Wrapper

Sebastien Huet — Fri, 08 Sep 2017 09:41:39 GMT

Hi @schef did you manage to write your wrapper? may you please share it? thx

RE: BLE C++ Wrapper

schef — Mon, 17 Jul 2017 08:06:48 GMT

Thanks for the anwser. I will have to see how i will solve this issue.

RE: BLE C++ Wrapper

Jørgen Holmefjord — Fri, 14 Jul 2017 11:41:58 GMT

Inside ble_advertising_start() there is a call to m_evt_handler(m_adv_evt):588. In advertising_init(), ble_advertising_init() is called with the parameter static_on_adv_evt, which you have implemented as follows:

void BLEWrapper::static_on_adv_evt(ble_adv_evt_t ble_adv_evt)
{
  /* static implementation of on_adv_evt for use with sdk callback function */
  BLEWrapper::getInstance()->on_adv_evt(ble_adv_evt);
}

Since your call to BLEWrapper::getInstance() from main.cpp have not completed, _pInstance have not yet been set, and the check in getInstance():

if(_pInstance == NULL)
{
  _pInstance = new BLEWrapper();
}

will evaluate to true. This will create an infinite loop of new BLEWrapper-instances, and softdevice initialization will fail with error code 8, since it is allready initialized.

You need to refactor your code to start advertising after init, or use another method of getting the instance.

Best regards,

Jørgen

RE: BLE C++ Wrapper

schef — Fri, 14 Jul 2017 09:53:31 GMT

in main.cpp:40 is a BLEWrapper::getInstance(); which should call the constructor. And constructor is in ble_wrapper.cpp:30 which calls init:609.

RE: BLE C++ Wrapper

Jørgen Holmefjord — Fri, 14 Jul 2017 09:50:10 GMT

Is the error in SOFTDEVICE_HANDLER_INIT, or from the call to ble_advertising_start? If the error is in SOFTDEVICE_HANDLER_INIT this error means that the stack is allready initialized, you should check that you are not calling this function multiple times. It does not make sense that you get the error in this function, but not if you comment out the proceeding call to ble_advertising_start. If the error is from ble_advertising_start, this means that the advertising module have not been successfully initialized. Where do you instantiate the class in you main.cpp file?