[PATCH] D93986: [clang-format] Add the possibility to align assignments spanning empty lines or comments

Björn Schäpers via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat Jan 16 06:24:24 PST 2021


HazardyKnusperkeks added a comment.

Not what I had in mind, but for me that's okay. I can not say anything to the change of the script though.

What I hat in mind was explaining (maybe in length) the enum and for the alignment settings just show an example with `None` and one of the other options, while referring to the enum documentation for all options.



================
Comment at: clang/include/clang/Format/Format.h:87
 
-  /// \brief If ``true``, aligns consecutive C/C++ preprocessor macros.
+  /// Styles for alignment of consecutive tokens. Tokens can be assignment signs
+  /// (see
----------------
If you don't document this with `///` you don't need to remove the error from the script, or am I mistaken?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D93986



More information about the cfe-commits mailing list