[PATCH] D28462: clang-format: Add new style option AlignConsecutiveMacros
Nick Renieris via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Mar 20 07:45:27 PDT 2019
VelocityRa updated this revision to Diff 191493.
VelocityRa marked 13 inline comments as done.
VelocityRa added a comment.
Diff updated.
Some comments about the inline changes, since the rest were trivial:
- `AlignMacroSequence` was inlined
- `AlignMacros` was after that
- `AlignMacrosMatches` can't be inlined because it's used both by `alignConsecutiveMacros`'s (pre-`AlignMacros`) `AlignCurrentSequence` lambda and `alignConsecutiveMacros` itself.
- `AlignCurrentSequence` can't be inlined because it's used three times in `alignConsecutiveMacros`.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D28462/new/
https://reviews.llvm.org/D28462
Files:
docs/ClangFormatStyleOptions.rst
include/clang/Format/Format.h
lib/Format/Format.cpp
lib/Format/WhitespaceManager.cpp
lib/Format/WhitespaceManager.h
unittests/Format/FormatTest.cpp
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D28462.191493.patch
Type: text/x-patch
Size: 12079 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20190320/2299eebc/attachment-0001.bin>
More information about the cfe-commits
mailing list