[PATCH] D93986: [clang-format] Add the possibility to align assignments spanning empty lines or comments
MyDeveloperDay via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Jan 5 02:34:51 PST 2021
MyDeveloperDay added inline comments.
================
Comment at: clang/docs/ClangFormatStyleOptions.rst:198
-**AlignConsecutiveAssignments** (``bool``)
- If ``true``, aligns consecutive assignments.
+**AlignConsecutiveAssignments** (``AlignConsecutiveAssignmentsStyle``)
+ Style of aligning assignments on consecutive lines.
----------------
As you plan to do the other cases won't this introduce
`AlignBitFieldStyle`
and
`AlignMacroStyle`
why not have 1 `AlignConsecutiveStyle` that can be used for all 3, then you don't need the other enum `TokenAlignmentStyle ` which means you won't need to map from one to the other?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D93986/new/
https://reviews.llvm.org/D93986
More information about the cfe-commits
mailing list