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

[Lukas Barth via Phabricator via cfe-commits]

tinloaf marked 3 inline comments as done.
tinloaf added inline comments.


================
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
----------------
HazardyKnusperkeks wrote:
> If you don't document this with `///` you don't need to remove the error from the script, or am I mistaken?
Unfortunately no: The dump script's state machine never goes into the `InEnum` state, and no `Enum` object with the correct name is created at all. This then breaks the doc generation for the fields using that enum. Sure, I could probably work around that, but that would probably be a more invasive change than silently ignoring undocumented enum members.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D93986