Skip to content

sys: replace header guards with #pragma once #21497

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
Merged
merged 2 commits into from
May 20, 2025
Merged

sys: replace header guards with #pragma once #21497

merged 2 commits into from
May 20, 2025

Conversation

N11cc00
Copy link
Contributor

@N11cc00 N11cc00 commented May 16, 2025 •
edited
Loading

Contribution description

This is part of a greater effort to replace all header guards in RIOT with #pragma once. This PR only affects all headers under sys/.

Issues/PRs references

Tracking issue #21335
Outdated, now closed #21340

@github-actions github-actions bot added Area: network Area: Networking Area: BLE Area: Bluetooth Low Energy support Area: timers Area: timer subsystems Area: arduino API Area: Arduino wrapper API Area: LoRa Area: LoRa radio support Area: SAUL Area: Sensor/Actuator Uber Layer Area: OTA Area: Over-the-air updates Area: CoAP Area: Constrained Application Protocol implementations Area: USB Area: Universal Serial Bus Area: sys Area: System labels May 16, 2025
@N11cc00 N11cc00 mentioned this pull request May 16, 2025
Open
8 tasks
@crasbe crasbe added Type: cleanup The issue proposes a clean-up / The PR cleans-up parts of the codebase / documentation CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR labels May 16, 2025
@riot-ci
Copy link

riot-ci commented May 16, 2025 •
edited
Loading

Murdock results

✔️ PASSED

9c6820b sys/include: define for XTIMER added

Success Failures Total Runtime
10346 0 10346 14m:22s

Artifacts

maribu
maribu reviewed May 18, 2025
View reviewed changes
maribu
maribu reviewed May 18, 2025
View reviewed changes
Copy link
Member

@maribu maribu left a comment

Choose a reason for hiding this comment

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

Thx for doing this!

The CI is unhappy, see inline for a suggestion on how to make it happy.

@N11cc00 N11cc00 force-pushed the fix-sys-headers branch from 79913b6 to af3c557 Compare May 19, 2025 11:58
@N11cc00 N11cc00 force-pushed the fix-sys-headers branch from af3c557 to 9c6820b Compare May 19, 2025 12:01
mguetschow
mguetschow approved these changes May 19, 2025
View reviewed changes
Copy link
Contributor

@mguetschow mguetschow left a comment

Choose a reason for hiding this comment

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

Great, thanks a lot!

@mguetschow mguetschow enabled auto-merge May 19, 2025 15:55
@mguetschow mguetschow added this pull request to the merge queue May 19, 2025
Merged via the queue into RIOT-OS:master with commit fd7f4e6 May 20, 2025
26 checks passed
@Teufelchen1 Teufelchen1 added this to the Release 2025.07 milestone Jul 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@maribu maribu maribu left review comments

@mguetschow mguetschow mguetschow approved these changes

@kaspar030 kaspar030 Awaiting requested review from kaspar030 kaspar030 is a code owner

@bergzand bergzand Awaiting requested review from bergzand bergzand is a code owner

@miri64 miri64 Awaiting requested review from miri64 miri64 is a code owner

@PeterKietzmann PeterKietzmann Awaiting requested review from PeterKietzmann PeterKietzmann is a code owner

@leandrolanzieri leandrolanzieri Awaiting requested review from leandrolanzieri leandrolanzieri is a code owner

Assignees
No one assigned
Labels
Area: arduino API Area: Arduino wrapper API Area: BLE Area: Bluetooth Low Energy support Area: CoAP Area: Constrained Application Protocol implementations Area: LoRa Area: LoRa radio support Area: network Area: Networking Area: OTA Area: Over-the-air updates Area: SAUL Area: Sensor/Actuator Uber Layer Area: sys Area: System Area: timers Area: timer subsystems Area: USB Area: Universal Serial Bus CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Type: cleanup The issue proposes a clean-up / The PR cleans-up parts of the codebase / documentation
Projects
None yet
Milestone
Release 2025.07
Development

Successfully merging this pull request may close these issues.
6 participants
@N11cc00 @riot-ci @maribu @mguetschow @crasbe @Teufelchen1