[PATCH] D68682: Clang-tidy fix removals removing all non-blank text from a line should remove the line
Alexander Kornienko via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Oct 14 01:16:20 PDT 2019
alexfh added a comment.
In D68682#1706581 <https://reviews.llvm.org/D68682#1706581>, @poelmanc wrote:
> Thanks, from the name that sounds like the perfect place to do it. If `cleanupAroundReplacements` also is used by clang-format, would we want to make the functionality optional, e.g. via a new bool parameter to `cleanupAroundReplacements`, a new option in FormatStyle, etc.?
cleanupAroundReplacements is not used by clang-format itself. I believe, it's a part of clang-format only because it was easier to implement it on top of some infrastructure clang-format provides.
Repository:
rCTE Clang Tools Extra
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D68682/new/
https://reviews.llvm.org/D68682
More information about the cfe-commits
mailing list