curdeius added a comment.
It seems to be a bug indeed.
So, basically all the changes are just adding a bunch of conditions on
`ColumnLimit != 0` which is a special value meaning "no limit", right? Or have
I missed something?
Thining out loud... wouldn't it be easier to change `ColumnLimit` from 0 to
`numeric_limits<unsigned>::max()` just before these transformations?
================
Comment at: clang/lib/Format/ContinuationIndenter.cpp:2243
Reflow = true;
if (ContentStartColumn + RemainingTokenColumns > ColumnLimit) {
LLVM_DEBUG(llvm::dbgs()
----------------
Is this condition on `ColumnLimit` (and other below) working ok when
`ColumnLimit == 0`? Do your tests cover it?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D96896/new/
https://reviews.llvm.org/D96896
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
