[clang] [clang-format] revert to string << string handling to previous default (PR #88490)

Björn Schäpers via cfe-commits cfe-commits at lists.llvm.org
Fri Apr 12 13:03:30 PDT 2024


================
@@ -5598,10 +5598,34 @@ bool TokenAnnotator::mustBreakBefore(const AnnotatedLine &Line,
   // 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 &&
-      Right.Next->is(tok::string_literal) && Left.is(tok::string_literal) &&
-      Left.TokenText.ends_with("\\n\"")) {
-    return true;
+  if (Style.BreakChevronOperator == FormatStyle::BCOS_BetweenStrings) {
----------------
HazardyKnusperkeks wrote:

I'd go for a `switch`.

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


More information about the cfe-commits mailing list