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
http://lists.llvm.org/cgi-bin/mailman/li
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
___
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: unittests/Format/FormatTestOb
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
=
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
unittests/Format/FormatTestOb