jtbandes added inline comments.
================ Comment at: lib/Format/ContinuationIndenter.cpp:923 + // Don't propagate AvoidBinPacking into subexpressions of arg/param lists. + if (Current.FakeLParens.size() > 0 && + Current.FakeLParens.back() > prec::Comma) { ---------------- jtbandes wrote: > djasper wrote: > > I think you cannot get here if .size() is 0 as this is iterating over the > > elements. > Good point, thanks. Actually, I can probably just use `*I > prec::Comma`. https://reviews.llvm.org/D32475 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits