[PATCH] D93846: [clang-format] PR16518 Add flag to suppress empty line insertion before access modifier
MyDeveloperDay via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Jan 1 10:15:55 PST 2021
MyDeveloperDay added inline comments.
================
Comment at: clang/lib/Format/UnwrappedLineFormatter.cpp:1221
+ if (Style.EmptyLineBeforeAccessModifier &&
+ PreviousLine->Last->isOneOf(tok::semi, tok::r_brace) &&
+ RootToken.NewlinesBefore == 1)
----------------
maybe I don't understand the logic but why is this `r_brace` and shouldn't we be looking at the "Last Non Comment" token?
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D93846/new/
https://reviews.llvm.org/D93846
More information about the cfe-commits
mailing list