Skip to content

Add rustfmt-like clang-format#39

Merged
contagon merged 2 commits into
masterfrom
easton/clang-format
Jul 18, 2025
Merged

Add rustfmt-like clang-format#39
contagon merged 2 commits into
masterfrom
easton/clang-format

Conversation

@contagon
Copy link
Copy Markdown
Owner

No description provided.

@contagon contagon requested a review from Copilot July 13, 2025 02:07
Copy link
Copy Markdown
Contributor

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 adds a Rustfmt-like clang-format configuration and reformats the existing C++ headers and binding files to match the style rules.

  • Introduce a .clang-format with prescribed style settings.
  • Convert single-line function and lambda definitions into consistent multi-line formatting.
  • Align chained Nanobind calls and include directives according to the new style.

Reviewed Changes

Copilot reviewed 9 out of 10 changed files in this pull request and generated no comments.

Show a summary per file
File Description
cpp/evalio/types.h Expanded inline methods and operators to multiline definitions.
cpp/evalio/pipeline.h Reformatted static info methods to multiline style.
cpp/bindings/types.h Wrapped Nanobind .def calls and includes onto separate lines.
cpp/bindings/ros_pc2.h Reformatted lambdas and function signatures to multiline.
cpp/bindings/pipelines/lio_sam.h Converted inline helpers and class methods to multiline.
cpp/bindings/pipelines/kiss_icp.h Updated pipeline helpers and methods to multiline definitions.
cpp/bindings/pipeline.h Reformatted trampoline and binding calls to multiline style.
cpp/bindings/main.cpp Adjusted module definitions to multiline formatting.
.clang-format Added new clang-format configuration file.

@contagon contagon merged commit 1189f5b into master Jul 18, 2025
12 checks passed
@contagon contagon deleted the easton/clang-format branch July 19, 2025 18:23
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.

2 participants