[PATCH] D47195: [clang-format] Fix ObjC message arguments handling
Jacek Olesiak via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue May 22 08:50:19 PDT 2018
jolesiak added inline comments.
================
Comment at: unittests/Format/FormatTestObjC.cpp:815
+
+ // No line break before closing receiver's scope.
+ verifyFormat("[[obj a:42] a:42\n"
----------------
krasimir wrote:
> What's the receiver's scope in this comment referring to?
> Also, how would the old test cases be formatted?
For a receiver: `[obj a:42]` I meant `]` as a token closing the scope.
I'll rephrase the comment to be more precise.
Old tests were introduced in D46879. After this change the formatting will be the same as it was before D46879, i.e. the same as for last test touched in this change:
```
[[obj aaaaaa:42
b:42]
cc:42
d:42];
```
even if
```
[[obj aaaaaa:42
b:42]
cc:42 d:42];
```
satisfies the column limit.
Repository:
rC Clang
https://reviews.llvm.org/D47195
More information about the cfe-commits
mailing list