[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