[PATCH] D154091: [clang-format] Prefer breaking long strings at new lines

[sstwcw via Phabricator via cfe-commits]

sstwcw added inline comments.


================
Comment at: clang/lib/Format/BreakableToken.cpp:225
+    return BreakableToken::Split(NewLine, 0);
+  if (AfterSpace >= 2)
+    return BreakableToken::Split(AfterSpace, 0);
----------------
Should I change the threshold from 2 to 1?

It is set to 2 because it is the old behavior.  There is already a test.

```
already in the test:
"some"
" tex"
"t"

I expect:
"some"
" "
"text"
```


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D154091