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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits