[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