Skip to content

Conversation

@thekurtovic
Copy link
Collaborator

No description provided.

@h2zero
Copy link
Owner

h2zero commented Jan 24, 2025

Nice, I've been meaning to do this for a long time, just never became a priority :) Thanks!

@thekurtovic
Copy link
Collaborator Author

Updated documentation to clarify return codes.

@h2zero
Copy link
Owner

h2zero commented Jan 26, 2025

The default callback should still call ble_store_util_status_rr to maintain the currently expected functionality.

@finger563
Copy link
Contributor

Thanks for implementing this, definitely helpful :)

@h2zero h2zero merged commit b29919d into h2zero:master Feb 5, 2025
59 checks passed
finger563 added a commit to esp-cpp/esp-nimble-cpp that referenced this pull request Feb 10, 2025
* Add missing definition for default device callbacks (which prevents calling the `setDeviceCallbacks` method)
* Ensure `m_pDeviceCallbacks` inital value is set to `&defaultDeviceCallbacks` to prevent crash when pairing table is full

After h2zero#295 any time the pairing table fills up, the device will crash on the next pairing attempt.
h2zero pushed a commit that referenced this pull request Feb 13, 2025
* Add missing definition for default device callbacks (which prevents calling the `setDeviceCallbacks` method)
* Ensure `m_pDeviceCallbacks` inital value is set to `&defaultDeviceCallbacks` to prevent crash when pairing table is full

After #295 any time the pairing table fills up, the device will crash on the next pairing attempt.
h2zero pushed a commit that referenced this pull request Feb 25, 2025
* Add missing definition for default device callbacks (which prevents calling the `setDeviceCallbacks` method)
* Ensure `m_pDeviceCallbacks` inital value is set to `&defaultDeviceCallbacks` to prevent crash when pairing table is full

After #295 any time the pairing table fills up, the device will crash on the next pairing attempt.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants