[PATCH] D101868: [clang-format] Adds a formatter for aligning arrays of structs

[Fred Grim via Phabricator via cfe-commits]

feg208 marked 6 inline comments as done.
feg208 added a comment.

I rolled up the suggested changes.



================
Comment at: clang/lib/Format/ContinuationIndenter.cpp:603
                                   State.Column + Spaces + PPColumnCorrection);
-
   // If "BreakBeforeInheritanceComma" mode, don't break within the inheritance
----------------
HazardyKnusperkeks wrote:
> Please don't remove the empty lines.
I am sorry. I was peeling back previous changes and must have overdone it


================
Comment at: clang/lib/Format/TokenAnnotator.cpp:2657-2659
+    if (CurrentToken->is(tok::l_brace)) {
+      ++Depth;
+    } else if (CurrentToken->is(tok::r_brace))
----------------
HazardyKnusperkeks wrote:
> There are still braces.
blech I thought I got those. Sorry.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D101868