sstwcw marked 18 inline comments as done. sstwcw added a comment. The YAML stuff are now in D120363 <https://reviews.llvm.org/D120363>.
================ Comment at: clang/docs/ClangFormatStyleOptions.rst:1408-1415 + Precise control over the wrapping of braces. + + .. code-block:: c++ + + # Should be declared this way: + BreakBeforeBraces: Custom + BraceWrapping: ---------------- curdeius wrote: > That's an unrelated change. Could you please do it in another (NFC) revision? It's in D120361. ================ Comment at: clang/docs/tools/dump_format_style.py:121 def __str__(self): - return '\n'.join(map(str, self.values)) + return self.comment + '\n' + '\n'.join(map(str, self.values)) ---------------- MyDeveloperDay wrote: > Can this change be separate? why is this needed? Could you add a screengrab > of the html that it generates? It's in D120361. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D119599/new/ https://reviews.llvm.org/D119599 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
