[PATCH] D30492: [clang-format] Allow all but the first string literal in a sequence to be put on a newline

2017-03-03 Thread Krasimir Georgiev via Phabricator via cfe-commits
krasimir abandoned this revision. krasimir added a comment. This patch is superseded by https://reviews.llvm.org/D30575. https://reviews.llvm.org/D30492 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D30492: [clang-format] Allow all but the first string literal in a sequence to be put on a newline

2017-03-02 Thread Daniel Jasper via Phabricator via cfe-commits
djasper added a comment. As discussed offline, I think this solves the wrong problem. My guess is that breakProtrudingToken checks State.Stack.back().NoLinebreak, but I forget to make it also check NoLinebreakInOperand. https://reviews.llvm.org/D30492

[PATCH] D30492: [clang-format] Allow all but the first string literal in a sequence to be put on a newline

2017-03-02 Thread Krasimir Georgiev via Phabricator via cfe-commits
krasimir updated this revision to Diff 90312. krasimir added a comment. - Reformat newly added chunk of code https://reviews.llvm.org/D30492 Files: lib/Format/ContinuationIndenter.cpp unittests/Format/FormatTest.cpp unittests/Format/FormatTestObjC.cpp Index:

[PATCH] D30492: [clang-format] Allow all but the first string literal in a sequence to be put on a newline

2017-03-02 Thread Krasimir Georgiev via Phabricator via cfe-commits
krasimir updated this revision to Diff 90310. krasimir added a comment. - Remove DEBUG statement https://reviews.llvm.org/D30492 Files: lib/Format/ContinuationIndenter.cpp unittests/Format/FormatTest.cpp unittests/Format/FormatTestObjC.cpp Index: unittests/Format/FormatTestObjC.cpp

[PATCH] D30492: [clang-format] Allow all but the first string literal in a sequence to be put on a newline

2017-03-02 Thread Krasimir Georgiev via Phabricator via cfe-commits
krasimir updated this revision to Diff 90309. krasimir added a comment. - Add missing test cases for commas and handling for Objective-C string literals https://reviews.llvm.org/D30492 Files: lib/Format/ContinuationIndenter.cpp unittests/Format/FormatTest.cpp