[clang] [clang-format][NFC] Upgrade IndentBraces option to a struct (PR #143663)
Owen Pan via cfe-commits
cfe-commits at lists.llvm.org
Sun Jun 15 18:25:00 PDT 2025
owenca wrote:
> I don't understand this fix, it needs tests to show what you are doing
See https://github.com/llvm/llvm-project/pull/143249#discussion_r2136966951 and https://github.com/llvm/llvm-project/pull/143249#issuecomment-2957775336. IMO, the current `BraceWrapping.IndentBraces` boolean is easy to understand: Indent (or not) the wrapped braces. I don't know if it makes sense to change it to an `enum` or `struct`, move it out of `BraceWrapping`, or add an `IndentLambdaBraces` to accommodate a very specific style in #143248.
https://github.com/llvm/llvm-project/pull/143663
More information about the cfe-commits
mailing list