[clang] [clang-format] Break after string literals with trailing line breaks (PR #76795)

Owen Pan via cfe-commits cfe-commits at lists.llvm.org
Fri Jan 5 01:45:45 PST 2024


================
@@ -5151,6 +5151,14 @@ bool TokenAnnotator::mustBreakBefore(const AnnotatedLine &Line,
     return true;
   if (Left.IsUnterminatedLiteral)
     return true;
+  // FIXME: Breaking after newlines seems useful in general. Turn this into an
+  // option and Recognize more cases like endl etc, and break independent of
+  // what comes after operator lessless.
+  if (Right.is(tok::lessless) && Right.Next && Left.is(tok::string_literal) &&
+      Right.Next->is(tok::string_literal) &&
----------------
owenca wrote:

```suggestion
  // option and recognize more cases like endl etc, and break independent of
  // what comes after operator lessless.
  if (Right.is(tok::lessless) && Right.Next &&
      Right.Next->is(tok::string_literal) && Left.is(tok::string_literal) &&
```

https://github.com/llvm/llvm-project/pull/76795


More information about the cfe-commits mailing list