benhamilton added inline comments.
================
Comment at: lib/Format/ContinuationIndenter.cpp:1214
// FIXME: We likely want to do this for more combinations of brackets.
// Verify that it is wanted for ObjC, too.
if (Current.is(tok::less) && Current.ParentBracket == tok::l_paren) {
----------------
stephanemoore wrote:
> With the new parameter does this comment still apply? Maybe we can remove it?
Good point.
This is about <> brackets nested inside () parens, which I believe is
applicable to Objective-C 2.0 generics passed to C/C++ functions, but not to
protocol conformance lists.
I'll send out a separate diff to add tests and clean up this comment, since I
think the behavior is wanted for ObjC as well, but I think it's unrelated to
this diff.
Repository:
rC Clang
https://reviews.llvm.org/D42650
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
