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

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to