sammccall added inline comments.
================ Comment at: clang-tools-extra/clangd/FormattedString.cpp:138 + // No need to pad from previous blocks, as they should end with a new line. + OS << renderCodeBlock(Contents, Language) << '\n'; + } ---------------- can you inline renderCodeBlock here, or modify it to just compute the marker? It doesn't make sense to have this function just to delegate to one with a different interface (and a copy) ================ Comment at: clang-tools-extra/clangd/FormattedString.cpp:142 + void renderPlainText(llvm::raw_ostream &OS) const override { + // In plaintext we want one empty line before and after codeblocks. + OS << '\n' << Contents << "\n\n"; ---------------- This means if we have two consecutive code blocks, they'll be separated by two blank lines... worth at least a fixme. (I guess the "right thing" here is to have `virtual unsigned verticalMargin() { return 0; }`, and have the document renderer insert max(first.verticalMargin(),second.verticalMargin()) newlines in between). Probably not worth it Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D71414/new/ https://reviews.llvm.org/D71414 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits