[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