[PATCH] D68682: Clang-tidy fixes should avoid creating new blank lines

Dmitri Gribenko via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Oct 9 01:42:56 PDT 2019


gribozavr added a reviewer: gribozavr.
gribozavr requested changes to this revision.
gribozavr added a comment.
This revision now requires changes to proceed.

+1 to what MyDeveloperDay said. The infrastructure can't know whether the newlines are intentional or not. Checks that edit the source code should be improved to delete newlines where they become unnecessary. I don't think we can accept the patch that changes how we apply edits.


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