Skip to content

libraries: Format with clang-format#173

Merged
soburi merged 1 commit intozephyrproject-rtos:nextfrom
soburi:lib_format
Apr 17, 2026
Merged

libraries: Format with clang-format#173
soburi merged 1 commit intozephyrproject-rtos:nextfrom
soburi:lib_format

Conversation

@soburi
Copy link
Copy Markdown
Member

@soburi soburi commented Apr 17, 2026

Use a formatter to standardize indentation.

Use a formatter to standardize indentation.

Signed-off-by: TOKITA Hiroshi <tokita.hiroshi@gmail.com>
Copilot AI review requested due to automatic review settings April 17, 2026 11:42
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR standardizes formatting in the Arduino compatibility Wire (I2C) and SPI libraries by applying clang-format, improving consistency with the repository’s .clang-format settings.

Changes:

  • Re-indented and reformatted Wire and SPI headers/implementations (tabs, spacing, line wrapping).
  • Expanded some previously single-line empty/short function bodies into multi-line brace blocks.
  • Adjusted formatting/alignment for a few macros and declarations.

Reviewed changes

Copilot reviewed 3 out of 4 changed files in this pull request and generated 1 comment.

File Description
libraries/Wire/Wire.h clang-format applied to class declarations, pointer spacing, and struct formatting
libraries/Wire/Wire.cpp clang-format applied to method definitions, indentation, and wrapping
libraries/SPI/SPI.h clang-format applied to indentation and macro alignment/wrapping
libraries/SPI/SPI.cpp clang-format applied to indentation and wrapping of SPI operations and helpers

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread libraries/SPI/SPI.cpp
@soburi soburi merged commit 78bcf6a into zephyrproject-rtos:next Apr 17, 2026
7 checks passed
@soburi soburi deleted the lib_format branch April 17, 2026 13:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants