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.

Sign up for GitHub

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

Jump to bottom

sys/frac/frac.c: add missing inttypes.h include #15480

Merged
merged 1 commit into from Nov 20, 2020
Merged

sys/frac/frac.c: add missing inttypes.h include #15480

merged 1 commit into from Nov 20, 2020

Conversation

kaspar030
Copy link
Contributor

Contribution description

The DEBUG() messages are using e.g., PRIu32. Those need inttypes.h. (I think they're included by some other header frac.c includes, but it should be explicit.)

Testing procedure

Issues/PRs references

@kaspar030 kaspar030 added Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors) Impact: minor The PR is small in size and might only require a quick look of a knowledgeable reviewer CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Area: sys Area: System labels Nov 20, 2020
bergzand
bergzand approved these changes Nov 20, 2020
View reviewed changes
Copy link
Member

@bergzand bergzand left a comment

Choose a reason for hiding this comment

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

ACK!

@benpicco benpicco merged commit dd1de91 into RIOT-OS:master Nov 20, 2020
@kaspar030 kaspar030 deleted the frac_inttypes branch November 26, 2020 14:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bergzand bergzand bergzand approved these changes

Assignees
No one assigned
Labels
Area: sys Area: System CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Impact: minor The PR is small in size and might only require a quick look of a knowledgeable reviewer Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors)
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@kaspar030 @bergzand @benpicco