krasimir added inline comments.
================ Comment at: lib/Format/ContinuationIndenter.cpp:1749 + } + if (!Reflow) { + // If we didn't reflow into the next line, the only space to consider is ---------------- nit: Maybe change this to `if (Reflow)` and switch the if-else bodies. ================ Comment at: lib/Format/ContinuationIndenter.cpp:1777 + assert(Penalty >= NewBreakPenalty); + Penalty -= NewBreakPenalty; + } ---------------- Shouldn't we be resetting `NewBreakBefore` to `false` here? https://reviews.llvm.org/D40310 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits