[PATCH] D140835: [clang-format] Improve UnwrappedLineParser::mightFitOnOneLine()

Owen Pan via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jan 5 13:32:14 PST 2023


owenpan added a comment.

In D140835#4028479 <https://reviews.llvm.org/D140835#4028479>, @HazardyKnusperkeks wrote:

> In D140835#4028274 <https://reviews.llvm.org/D140835#4028274>, @owenpan wrote:
>
>> If `RemoveBracesLLVM` is false, the line is too long and will wrap.
>
> That's what I fail to see, but if you say so.

The line is 24 characters long, but `ColumnLimit` is set to 20 on line 731.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D140835/new/

https://reviews.llvm.org/D140835



More information about the cfe-commits mailing list