[PATCH] D129443: [clang-format] Add option for aligning requires clause body
Björn Schäpers via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Oct 12 12:30:03 PDT 2022
HazardyKnusperkeks added inline comments.
================
Comment at: clang/docs/ReleaseNotes.rst:540
------------
-- Add `RemoveSemicolon` option for removing `;` after a non-empty function definition.
+- Add ``RemoveSemicolon`` option for removing ``;`` after a non-empty function definition.
+- Add ``RequiresExpressionIndentation`` option for configuring the alignment of requires-expressions.
----------------
Unrelated.
================
Comment at: clang/lib/Format/Format.cpp:1304
LLVMStyle.RequiresClausePosition = FormatStyle::RCPS_OwnLine;
+ LLVMStyle.RequiresExpressionIndentation = FormatStyle::REI_Keyword;
LLVMStyle.SeparateDefinitionBlocks = FormatStyle::SDS_Leave;
----------------
owenpan wrote:
> Delete it.
Why delete?
Set it to OuterScope.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D129443/new/
https://reviews.llvm.org/D129443
More information about the cfe-commits
mailing list