HazardyKnusperkeks added a comment. In D93986#2495561 <https://reviews.llvm.org/D93986#2495561>, @MyDeveloperDay wrote:
> I think I would remove the code examples from the "AlignConsecutive style" to > avoid confusion (that would be the first change) > > then perhaps regenerate and update the documentation so we can then see, its > seems most have a "code block", can the specific examples not go into that > area? > > And just have the enum explain its different configuration options? > > /// \brief If ``true``, aligns consecutive C/C++ preprocessor macros. > /// > /// This will align C/C++ preprocessor macros of consecutive lines. > /// Will result in formattings like > /// \code > /// #define SHORT_NAME 42 > /// #define LONGER_NAME 0x007f > /// #define EVEN_LONGER_NAME (2) > /// #define foo(x) (x * x) > /// #define bar(y, z) (y + z) > /// \endcode > bool AlignConsecutiveMacros; I think that's a good way to go. 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
