[clang-tools-extra] [clangd] Fix regression regarding new line handling for hover/signature help content (PR #162029)
kadir çetinkaya via cfe-commits
cfe-commits at lists.llvm.org
Tue Oct 28 01:56:22 PDT 2025
================
@@ -575,64 +631,18 @@ bool Paragraph::isHardLineBreakIndicator(llvm::StringRef Rest) const {
return false;
}
-bool Paragraph::isHardLineBreakAfter(llvm::StringRef Line,
- llvm::StringRef Rest) const {
- // In Markdown, 2 spaces before a line break forces a line break.
- // Add a line break for plaintext in this case too.
+bool Paragraph::isHardLineBreakAfter(llvm::StringRef Line, llvm::StringRef Rest,
+ bool IsMarkdown) const {
// Should we also consider whether Line is short?
- return Line.ends_with(" ") || punctuationIndicatesLineBreak(Line) ||
- isHardLineBreakIndicator(Rest);
+ return (punctuationIndicatesLineBreak(Line, IsMarkdown) ||
----------------
kadircet wrote:
nit: drop surrounding parenthesis
https://github.com/llvm/llvm-project/pull/162029
More information about the cfe-commits
mailing list