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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
