[PATCH] D40310: Restructure how we break tokens.

Manuel Klimek via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Nov 23 08:34:02 PST 2017


klimek marked an inline comment as done.
klimek added inline comments.


================
Comment at: lib/Format/ContinuationIndenter.cpp:1504
                                  : Style.PenaltyBreakComment;
-  unsigned RemainingSpace = ColumnLimit - Current.UnbreakableTailLength;
+  // Stores whether we introduce a break anywhere in the token.
   bool BreakInserted = Token->introducesBreakBeforeToken();
----------------
krasimir wrote:
> klimek wrote:
> > krasimir wrote:
> > > Does a reflow count as a break?
> > I do believe so (well, the break in the reflow counts, the reflow itself is not a break, but removing a break :)
> So, if we add a break then remove a break while reflowing, will `BreakInserted` be true or false?
True.


https://reviews.llvm.org/D40310





More information about the cfe-commits mailing list