[PATCH] D154091: [clang-format] Recognize escape sequences when breaking strings

Tue Jul 4 16:38:42 PDT 2023

sstwcw marked an inline comment as done.
sstwcw added a comment.

In D154091#4470158 <https://reviews.llvm.org/D154091#4470158>, @MyDeveloperDay wrote:

> What would happen in a \r\n example

If the entire `\r\n` part fits on the first line, the break is after `\r\n`.  If only the part up to `\r` fits on the first line, the break is between `\r` and `\n`.

================
Comment at: clang/lib/Format/BreakableToken.cpp:207
+      case 't':
+      case 'v':
+        AfterSpace = SplitPoint + 2;
----------------
MyDeveloperDay wrote:
> Are you testing \r \v \f
I am now.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D154091