[PATCH] D74669: [clang-tidy] New check: misc-no-include-cpp

Nathan James via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Feb 25 17:48:33 PST 2020


njames93 added a comment.

In D74669#1892577 <https://reviews.llvm.org/D74669#1892577>, @jroelofs wrote:

> Implement review feedback:
>
> - Re-purpose `HeaderFileExtensionsUtils.h` to support headers *and* sources.
> - Surround file extension in diagnostic with `''`s.
>
> I have these as two separate patches locally, but I don't know how to represent that in phabricator, so I've uploaded the diff across both. If this gets approved, I intend to push them as separate commits.


You can create a review that just re-purposes HeaderFileExtensionsUtils then set that review as a parent of this review


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D74669/new/

https://reviews.llvm.org/D74669





More information about the cfe-commits mailing list