[PATCH] D93986: [clang-format] Add the possibility to align assignments spanning empty lines or comments

Lukas Barth via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat Jan 16 04:33:18 PST 2021


tinloaf updated this revision to Diff 317166.
tinloaf added a comment.

Fixed documentation for the `AlignConsecutive*` options. I have used the suggested approach: Remove all documentation from `AlignConsecutiveStyle` and instead put the documentation of the possible values and what they do in the `AlignConsecutive*` options' docs.

I had to make two small changes to the documentation dump script for this:

- Non-documented options in enums are silently ignored instead of raising an error, since `AlignConsecutiveStyle` has non-documented options now.
- Indentation in the `code-block` lines is preserved, so that we can have indented code blocks in the documentation of the possible values.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D93986

Files:
  clang/docs/ClangFormatStyleOptions.rst
  clang/docs/ReleaseNotes.rst
  clang/docs/tools/dump_format_style.py
  clang/include/clang/Format/Format.h
  clang/lib/Format/Format.cpp
  clang/lib/Format/WhitespaceManager.cpp
  clang/unittests/Format/FormatTest.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D93986.317166.patch
Type: text/x-patch
Size: 69723 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20210116/0099d779/attachment-0001.bin>


More information about the cfe-commits mailing list