[PATCH] D54881: [clang-format] Prevent Clang-Format from editing leading whitespace on lines outside of the format range
Manuel Klimek via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Mar 28 10:07:23 PDT 2019
klimek added a comment.
I just verified why this doesn't break for our CI:
The trick is that if the indent is actually off (as opposed to being correct, but tab vs spaces), we do re-indent until we find an indent that's correct.
The fix that would make me happier, I think, is to do the same thing here:
Basically, when we expand the range until nothing changes, do not only check whether the indent is correct, but also whether we'd change the spaces in the indent.
WDYT?
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D54881/new/
https://reviews.llvm.org/D54881
More information about the cfe-commits
mailing list