[PATCH] D117520: [clang-format] Fix SeparateDefinitionBlocks issues

ksyx via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Jan 18 17:39:12 PST 2022


ksyx updated this revision to Diff 401062.
ksyx marked 2 inline comments as done.
ksyx edited the summary of this revision.
ksyx added a comment.

- Resolves formatting conflict with options `EmptyLineAfterAccessModifier` and `EmptyLineBeforeAccessModifier` (prompts with `--dry-run` (`-n`) or `--output-replacement-xml` but no observable change)
- Remove redundant and repeated initialization
- Do no change to newlines before EOF


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

https://reviews.llvm.org/D117520

Files:
  clang/lib/Format/DefinitionBlockSeparator.cpp
  clang/lib/Format/DefinitionBlockSeparator.h
  clang/unittests/Format/DefinitionBlockSeparatorTest.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D117520.401062.patch
Type: text/x-patch
Size: 14429 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20220119/3a83c69b/attachment-0001.bin>


More information about the cfe-commits mailing list