Skip to content

Format all C source code using clang-format#419

Merged
jasonkatonica merged 1 commit intoIBM:java17from
jasonkatonica:katonica/feature/clangformatwhitespace17
Jan 16, 2025
Merged

Format all C source code using clang-format#419
jasonkatonica merged 1 commit intoIBM:java17from
jasonkatonica:katonica/feature/clangformatwhitespace17

Conversation

@jasonkatonica
Copy link
Member

@jasonkatonica jasonkatonica commented Jan 15, 2025

This update adds a new .clang-format file that can be used to as a configuration file for the clang-format tool used to format the C code contained in this project. Code is formatted from both a stylistic and whitespace perspective.

The README.md was updated to describe the rules and the tooling being used to format the C code.

All .c and .h files within the project were formatted according to the rules contained in the .clang-format file.

Additional options can be found in the documentation link found at: https://clang.llvm.org/docs/ClangFormatStyleOptions.html

Back-ported from: #409

Signed-off-by: Jason Katonica katonica@us.ibm.com

This update adds a new `.clang-format` file that can be used to
as a configuration file for the clang-format tool used to format the
`C` code contained in this project. Code is formatted from both a
stylistic and whitespace perspective.

The `README.md` was updated to describe the rules and the tooling being
used to format the `C` code.

All `.c` and `.h` files within the project were formatted according
to the rules contained in the `.clang-format` file.

Additional options can be found in the documentation link found at:
https://clang.llvm.org/docs/ClangFormatStyleOptions.html

Signed-off-by: Jason Katonica <katonica@us.ibm.com>
Copy link
Collaborator

@taoliult taoliult left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@KostasTsiounis KostasTsiounis left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Collaborator

@JinhangZhang JinhangZhang left a comment

Choose a reason for hiding this comment

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

LGTM

@jasonkatonica jasonkatonica merged commit a377291 into IBM:java17 Jan 16, 2025
3 checks passed
@jasonkatonica jasonkatonica deleted the katonica/feature/clangformatwhitespace17 branch January 16, 2025 18:26
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.

5 participants