[PATCH] D66087: [clangd] Preserve line break when rendering text chunks of markdown

Sam McCall via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Aug 12 07:38:38 PDT 2019


sammccall added a comment.

Added a bit of a braindump to https://github.com/clangd/clangd/issues/95

I think always forcing a break is maybe at least as bad as forcing no break, and we should aim to be cleverer.
Some (relatively) simple punctuation/capitalization/line-length based heuristics may get us a long way here - is it ok to punt on this for a while?

This change would be consistent with how we handle plaintext, but including the breaks seems less problematic there: it's not clear editors are supposed to rewrap plaintext if we unwrap it, whereas markdown is very clear about that.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D66087





More information about the cfe-commits mailing list