[PATCH] D93846: [clang-format] PR16518 Add flag to suppress empty line insertion before access modifier

[MyDeveloperDay via Phabricator via cfe-commits]

MyDeveloperDay added inline comments.


================
Comment at: clang/lib/Format/UnwrappedLineFormatter.cpp:1222-1223
+        PreviousLine->Last->isOneOf(tok::semi, tok::r_brace) &&
+        RootToken.NewlinesBefore == 1)
+      ++Newlines;
+    else if (!Style.EmptyLineBeforeAccessModifier &&
----------------
HazardyKnusperkeks wrote:
> A tab?
My experience is that this doesn't mean a tab but just the what phabricator shows a change in whitespace

it is now 2 levels  of `if` scope indented not 1 so I think it should be moved over abit






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

https://reviews.llvm.org/D93846