[PATCH] D42372: [clang-format] Ignore UnbreakableTailLength sometimes during breaking

[Daniel Jasper via Phabricator via cfe-commits]

djasper added a comment.

I think I understand now. I think I'd prefer pulling all of the "+ UnbreakableTailLength" calculations out of getRemainingLength so that you don't have to pass around the new parameter CanBreakAfter. Instead, only add it if necessary outside of the function.



================
Comment at: lib/Format/BreakableToken.cpp:503
+                                                   bool CanBreakAfter) const {
   return UnbreakableTailLength +
          getRangeLength(LineIndex, Offset, StringRef::npos, StartColumn);
----------------
Forgot to check CanBreakAfter here?


Repository:
  rC Clang

https://reviews.llvm.org/D42372