[PATCH] D68296: [clang-format] Add ability to wrap braces after multi-line control statements
MyDeveloperDay via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Oct 2 12:04:40 PDT 2019
MyDeveloperDay added a comment.
@klimek thanks for offering for me to make the decision. I think given the fact that blender has it in their style guide and that they are clearly working around clang-formats inability to handle this case, I'm minded to LGTM this revision
I hope you are ok with that. given that @mitchell-stellar is prepared to support it, and I'm happy enough that I could fix something with that I hope you don't mind.
@mitchell-stellar are you happy with the name `OnlyMultiLine`? the reason I ask is:
`AlwaysBreakBeforeMultilineStrings` uses just `MultiLine`
> BTDS_MultiLine (in configuration: MultiLine) Force break after template declaration only when the following declaration spans multiple lines.
I think given this won't be a standard true/false parameter users having to remember the "Only" part seems unnecessary
Repository:
rC Clang
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D68296/new/
https://reviews.llvm.org/D68296
More information about the cfe-commits
mailing list