djasper added inline comments.
================
Comment at: lib/Format/TokenAnnotator.cpp:2009
+ // Prefer breaking call chains (".foo") over empty "{}", "[]" or "()".
+ if ((Left.is(tok::l_brace) && Right.is(tok::r_brace)) ||
+ (Left.is(tok::l_square) && Right.is(tok::r_square)) ||
----------------
Or just Left.opensScope() && Right.closesScope()?
Technically, that's probably not the same thing, but otherwise you have
unmatched brackets and that's bad no matter what, no?
https://reviews.llvm.org/D36139
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
