Skip to content

.clang-format: minor tweaks#22319

Open
maribu wants to merge 1 commit into
RIOT-OS:masterfrom
maribu:clang-format/tweaks
Open

.clang-format: minor tweaks#22319
maribu wants to merge 1 commit into
RIOT-OS:masterfrom
maribu:clang-format/tweaks

Conversation

@maribu
Copy link
Copy Markdown
Member

@maribu maribu commented May 21, 2026

Contribution description

  • adjust penalties to better match current code style
  • abuse SpaceBeforeCpp11BracedList to fix formation of NANOCOAP_RESOURCE macro.

Testing procedure

Run clang-format e.g. on examples/networking/coap/nanocoap_server/coap_handler.c. It should no longer remove the space between NANOCOAP_RESOURCE(echo) and {.

Issues/PRs references

None

Declaration of AI-Tools / LLMs usage:

AI-Tools / LLMs that were used are:

  • none

- adjust penalties to better match current code style
- abuse SpaceBeforeCpp11BracedList to fix formation of NANOCOAP_RESOURCE
  macro.
@maribu maribu added 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 labels May 21, 2026
@maribu maribu enabled auto-merge May 21, 2026 12:57
@riot-ci
Copy link
Copy Markdown

riot-ci commented May 21, 2026

Murdock results

✔️ PASSED

015767a .clang-format: minor tweaks

Success Failures Total Runtime
1 0 1 01m:19s

Artifacts

@crasbe crasbe added the Type: enhancement The issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation label May 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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: enhancement The issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants