[PATCH] D98237: [clang-format] Option for empty lines after an access modifier.

[Björn Schäpers via Phabricator via cfe-commits]

HazardyKnusperkeks added a comment.

I would like additional tests:

- With at least 2 empty lines
- With 0-2 empty lines without `verifyFormat` to demonstrate what is kept, added, and removed.



================
Comment at: clang/lib/Format/UnwrappedLineFormatter.cpp:1284
       (!PreviousLine->InPPDirective || !RootToken.HasUnescapedNewline))
-    Newlines = std::min(1u, Newlines);
+    Newlines = Style.EmptyLinesAfterAccessModifier + 1u;
 
----------------
I don't know, I'm just asking:
Shouldn't this be `Newlines = std::min(Newlines, Style.EmptyLinesAfterAccessModifier + 1u);`?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D98237