[PATCH] D119599: [clang-format] Add option to align compound assignments like `+=`
sstwcw via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Mar 4 15:10:07 PST 2022
sstwcw marked 7 inline comments as done.
sstwcw added inline comments.
================
Comment at: clang/docs/tools/dump_format_style.py:293
state = State.InNestedStruct
- field_type, field_name = re.match(r'([<>:\w(,\s)]+)\s+(\w+);', line).groups()
+ field_type, field_name = re.match(r'([<>:\w(,\s)]+)\s+(\w+)', line).groups()
if field_type in enums:
----------------
This change is needed to parse the entry `bool PadOperators = true;`.
================
Comment at: clang/lib/Format/WhitespaceManager.cpp:273
+// RightJustify - Whether it is the token's right end or left end that
+// gets moved to that column.
template <typename F>
----------------
MyDeveloperDay wrote:
> something odd here why not, the following its fits 80 columns
>
> ```
> // Column - The token for which Matches returns true is moved to this column.
> // RightJustify - Whether it is the token's right end or left end that gets
> // moved to that column.
> ```
My text editor automatically limited comments to 72 columns.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D119599/new/
https://reviews.llvm.org/D119599
More information about the llvm-commits
mailing list