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

gnrc_sixlowpan_frag_vrb: fix cppcheck warning [backport 2020.07] #14686

Conversation

miri64
Copy link
Member

@miri64 miri64 commented Aug 3, 2020

Backport of #14683

Contribution description

... which wasn't a false positive after all. See also #12848 (review)

Testing procedure

make -C tests/gnrc_sixlowpan_iphc_w_vrb flash test

and

make -C tests/unittests tests-gnrc_sixlowpan_frag_vrb flash-only test

should still succeed for any supported board of choice.

Issues/PRs references

See #12848 (review)

... which wasn't a false positive after all.

(cherry picked from commit 5885293)
@miri64 miri64 added Area: network Area: Networking CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Process: release backport Integration Process: The PR is a release backport of a change previously provided to master Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors) labels Aug 3, 2020
Copy link
Contributor

@benpicco benpicco left a comment

Choose a reason for hiding this comment

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

Change is identical.

@miri64 miri64 merged commit f4580e8 into RIOT-OS:2020.07-branch Aug 3, 2020
@miri64 miri64 deleted the backport/2020.07/gnrc_sixlowpan_frag_vrb/fix/cppcheck-warning branch August 3, 2020 13:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: network Area: Networking CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Process: release backport Integration Process: The PR is a release backport of a change previously provided to master Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants