Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

BLEAddress: Correct less- and greater-than operators #6521

Merged
merged 1 commit into from
Apr 4, 2022
Merged

BLEAddress: Correct less- and greater-than operators #6521

merged 1 commit into from
Apr 4, 2022

Conversation

BrianRossmajer
Copy link
Contributor

By completing this PR sufficiently, you help us to improve the quality of Release Notes

Checklist

Summary

Reversed the arguments to memcmp so that eg 'a4:...' > 'a3:...' is true

Impact

Lets Manufacturer ranges in bluetooth address space be filtered correctly

Related links

@CLAassistant
Copy link

CLAassistant commented Apr 2, 2022

CLA assistant check
All committers have signed the CLA.

@github-actions
Copy link
Contributor

github-actions bot commented Apr 4, 2022

Unit Test Results

0 files  0 suites   0s ⏱️
0 tests 0 ✔️ 0 💤 0

Results for commit 993473a.

@me-no-dev me-no-dev merged commit 6fe1c4c into espressif:master Apr 4, 2022
@me-no-dev
Copy link
Member

Thanks @BrianRossmajer for the fix!

@BrianRossmajer BrianRossmajer deleted the patch-1 branch April 4, 2022 14:49
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.

None yet

3 participants