[PATCH] D119599: [clang-format] Add option to align compound assignments like `+=`

Björn Schäpers via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat Mar 5 05:12:32 PST 2022


HazardyKnusperkeks 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:
----------------
sstwcw wrote:
> This change is needed to parse the entry `bool PadOperators = true;`.
That is a sign that you should keep consistency and initialize `PadOperators` like the rest.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D119599/new/

https://reviews.llvm.org/D119599



More information about the cfe-commits mailing list