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

MyDeveloperDay via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jun 9 00:55:00 PDT 2021


MyDeveloperDay added inline comments.


================
Comment at: clang/lib/Format/WhitespaceManager.h:233
 
+  /// Align Array Initializers over all \c Changes
+  void alignArrayInitializers();
----------------
Can ensure the comment starts with a capital and finishes with punctuation (I personally don't mind but I believe that is the convention)


================
Comment at: clang/lib/Format/WhitespaceManager.h:274
+
+  /// Get the maximum width of a cell in a sequence of columns
+  template <typename I>
----------------



================
Comment at: clang/lib/Format/WhitespaceManager.h:291
+
+  /// Get The maximum width of all columns to a given cell
+  template <typename I>
----------------



================
Comment at: clang/lib/Format/WhitespaceManager.h:310
+
+  /// align a split cell with a newline to the first element in the cell
+  void alignToStartOfCell(unsigned Start, unsigned End);
----------------



================
Comment at: clang/lib/Format/WhitespaceManager.h:313
+
+  /// Link the Cell pointers in the list of Cells
+  static CellDescriptions linkCells(CellDescriptions &&CellDesc);
----------------



Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D101868



More information about the cfe-commits mailing list