Skip to content

Conversation

@sjanc
Copy link
Contributor

@sjanc sjanc commented Dec 12, 2025

If CEP is used, but no CCD than expected handle may vary. Instead of hardcoded value use variable for tracking expected next handle.

@sjanc
Copy link
Contributor Author

sjanc commented Dec 12, 2025

@piotrnarajowski fyi

@github-actions github-actions bot added the size/XS Extra small PR label Dec 12, 2025
Copy link
Contributor

@piotrnarajowski piotrnarajowski left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

If CEP is used, but no CCD than expected handle may vary. Instead of
hardcoded value use variable for tracking expected next handle.
@sjanc sjanc force-pushed the gatts_expected_handle branch from 4c494a0 to ec73f14 Compare December 12, 2025 15:33
@github-actions github-actions bot added the size/S Small PR label Dec 12, 2025
@sjanc sjanc changed the title nimble/ll: Fix assert when registering GATT server characteristics nimble/host: Fix assert when registering GATT server characteristics Dec 12, 2025
@sjanc sjanc merged commit ceaed8e into apache:master Dec 12, 2025
22 checks passed
@sjanc sjanc deleted the gatts_expected_handle branch December 12, 2025 15:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

host size/S Small PR size/XS Extra small PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants