[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