Skip to content
New issue

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

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

Already on GitHub? # to your account

sys/string_utils: add memchk() #19686

Merged
merged 4 commits into from
May 31, 2023
Merged

sys/string_utils: add memchk() #19686

merged 4 commits into from
May 31, 2023

Conversation

benpicco
Copy link
Contributor

Contribution description

This adds a function to check if all bytes in a buffer are set to a certain value.
Currently only gnrc_pktbuf_static` makes use of this, but I've used something similar in other places, often open-coded - so let's add it in a common place instead.

Testing procedure

A new unit test has been added.

Issues/PRs references

@benpicco benpicco requested a review from maribu May 30, 2023 18:48
@github-actions github-actions bot added Area: network Area: Networking Area: sys Area: System Area: tests Area: tests and testing framework labels May 30, 2023
@benpicco benpicco added the CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR label May 30, 2023
@riot-ci
Copy link

riot-ci commented May 30, 2023

Murdock results

✔️ PASSED

4476b2a sys/net/gnrc_pktbuf: make use of memchk()

Success Failures Total Runtime
6933 0 6933 16m:55s

Artifacts

@maribu
Copy link
Member

maribu commented May 30, 2023

bors merge

@bors
Copy link
Contributor

bors bot commented May 31, 2023

Build succeeded!

The publicly hosted instance of bors-ng is deprecated and will go away soon.

If you want to self-host your own instance, instructions are here.
For more help, visit the forum.

If you want to switch to GitHub's built-in merge queue, visit their help page.

@bors bors bot merged commit 67c011f into RIOT-OS:master May 31, 2023
@benpicco benpicco deleted the mem_is_set branch May 31, 2023 09:57
@benpicco
Copy link
Contributor Author

Thank you, that was quick :D

@benpicco benpicco added this to the Release 2023.07 milestone Aug 2, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Area: network Area: Networking Area: sys Area: System Area: tests Area: tests and testing framework CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants