[PATCH] D109557: Adds an AlignCloseBracket option
Cameron Mulhern via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Sep 13 16:38:54 PDT 2021
csmulhern added a comment.
In D109557#2998213 <https://reviews.llvm.org/D109557#2998213>, @HazardyKnusperkeks wrote:
> With context he meant the diff context. https://llvm.org/docs/Phabricator.html#requesting-a-review-via-the-web-interface
Ah sorry about that. Done.
In D109557#2998226 <https://reviews.llvm.org/D109557#2998226>, @HazardyKnusperkeks wrote:
> You state in the documentation that it is also for angle brackets and more, but there are no test cases for that.
Yeah, I wasn't sure exactly how to deal with this. The default behavior is already to align angle brackets and braces on newlines. See: https://github.com/llvm/llvm-project/blob/8a780a2f18c590e27e51a2ab3cc81b481c42b42a/clang/lib/Format/ContinuationIndenter.cpp#L341 (BreakBeforeClosingBrace is true when the block was started with a newline). Thus, you're already getting this behavior when CBAS_AlwaysBreak is set. I didn't want to make DontAlign (the default) explicitly opt out of this behavior. I guess we can narrow the scope of CloseBracketAlignmentStyle to just parenthesis, but that doesn't feel great either. What are your thoughts?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D109557/new/
https://reviews.llvm.org/D109557
More information about the cfe-commits
mailing list