[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