[PATCH] D119067: [clang-format] Fix DefinitionBlockSeparator extra empty lines
ksyx via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Sat Feb 5 18:29:17 PST 2022
ksyx created this revision.
ksyx added reviewers: MyDeveloperDay, curdeius, HazardyKnusperkeks.
ksyx added a project: clang-format.
ksyx requested review of this revision.
Herald added a project: clang.
Herald added a subscriber: cfe-commits.
This commit fixes https://github.com/llvm/llvm-project/issues/53229, in
which keywords like class and struct in a line ending with left brace or
whose next line is left brace only, will be falsely recognized as
definition line, causing extra empty lines inserted surrounding blocks
with no need to be formatted.
Repository:
rG LLVM Github Monorepo
https://reviews.llvm.org/D119067
Files:
clang/lib/Format/DefinitionBlockSeparator.cpp
clang/unittests/Format/DefinitionBlockSeparatorTest.cpp
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D119067.406222.patch
Type: text/x-patch
Size: 4399 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20220206/61405ad2/attachment-0001.bin>
More information about the cfe-commits
mailing list